Reorganization of Gaudi::Parsers code (!759) · Merge requests · Gaudi / Gaudi · GitLab

Snippets Groups Projects

Merged Marco Clemencic requested to merge clemenci/Gaudi:reorganize-parsers-code into master 6 years ago

Following up on a suggestion by @flemaitr, and because needed by !458, I fixed the issue with the strict requirement on order of inclusions of parsers headers.

The changes ar backward compatible except for some very rare cases.

moved related headers to different hierarchy
- Gaudi/Parsers/XYZ.h
enable ADL for calls to Gaudi::Parsers::parse
- use a helper class equivalent to std::string (Gaudi::Parsers::InputData)

Edited 6 years ago by Marco Clemencic

Activity

Marco Clemencic added 29 commits 6 years ago
added 29 commits

8103d75a - Fixed use of property in ViewTester

f34dc515 - Make new PropertyHolder methods public

9cd71268 - Add macro to detect presence of Gaudi::Interfaces::IOptionsSvc

c9cae8a6 - Fix small typo in THistSvc

8924113d - Bind properties to entries in options service

df56b387 - Add warning about unused properties set in options

b9eadbca - Allow declaration of defaults for options matching a regular expression

048dc71a - Use string_view instead of boost::string_ref

0abf5441 - Prototype of a new interface for logging

18253fc5 - Applied @graven suggestions

8d2197bb - Modified helper for iostreams formatting

d29622e2 - Moved Logging formatters to dedicated header

17a490aa - Made fmtlib available to Gaudi

23c03756 - Use fmtlib in Gaudi::Logging

2656a12a - Minor implementation clean up

cde95152 - Removed LoggingTraits and avoid level filtering in Logging::log

1cf2da5c - Modified Logging to accept a formatter instance or a callable

afce43c4 - Improved Logging interface

e0206c40 - Improve performance for non-printed messages

024f25fb - Reorganized Logging namespace

93a7183a - Reorganization of Logging headers and code

e73431da - Add property OutputLevel to Gaudi::Logging::Logger

5bff397c - Use same numeric level for MSG:Level and Gaudi::Logging::Level

06ddbe81 - Dropped `Link` in favour of `std::weak_ptr`

af1ee569 - Added Gaudi::Logging::Context

b3dbbea2 - Do not rely on SmartIF<IProperty>(serviceLocator()) being ApplicationMgr

0065a136 - Concurrent use of CommonMessaging and Gaudi::Logging::Logger

8aeb0766 - Make SvcLocator IProperty point to ApplicationMgr (backward compatibility hack)

88049cc1 - Reorganization of Gaudi::Parsers code

Compare with previous version
Toggle commit list
Marco Clemencic added 1 commit 6 years ago
added 1 commit

98bc84ac - Reorganization of Gaudi::Parsers code

Compare with previous version
Marco Clemencic changed the description 6 years ago

changed the description
Marco Clemencic unmarked as a Work In Progress 6 years ago

unmarked as a Work In Progress
Marco Clemencic marked as a Work In Progress 6 years ago

marked as a Work In Progress
L
Software for LHCb @lhcbsoft · 6 years ago
[2018-09-23 09:27] Validation started with lhcb-gaudi-merge#585

[2018-09-23 15:20] Validation started with lhcb-gaudi-merge#585

[2018-09-24 12:07] Validation started with lhcb-gaudi-merge#587

[2018-09-24 15:55] Validation started with lhcb-gaudi-merge#588

[2018-09-25 00:17] Validation started with lhcb-lcg-dev4#663

[2018-09-25 00:21] Validation started with lhcb-tdr-test#299

[2018-09-25 00:22] Validation started with lhcb-sanitizers#14

[2018-09-25 00:24] Validation started with lhcb-lcg-dev3#659

[2018-09-25 00:30] Validation started with lhcb-dd4hep#7

[2018-09-26 00:08] Validation started with lhcb-lcg-dev3#660

[2018-09-26 00:11] Validation started with lhcb-lcg-dev4#664

[2018-09-26 00:15] Validation started with lhcb-gaudi-head#2007

[2018-09-26 00:16] Validation started with lhcb-tdr-test#300

[2018-09-27 00:06] Validation started with lhcb-lcg-dev4#665

[2018-09-27 00:08] Validation started with lhcb-lcg-dev3#661

[2018-09-27 00:09] Validation started with lhcb-dd4hep#9

[2018-09-27 00:10] Validation started with lhcb-gaudi-head#2008

[2018-09-27 00:12] Validation started with lhcb-tdr-test#301

[2018-09-27 00:12] Validation started with lhcb-sanitizers#16

[2018-09-27 01:08] Validation started with lhcb-tdr-test#301

[2018-09-28 00:32] Validation started with lhcb-lcg-dev4#666

[2018-09-28 00:34] Validation started with lhcb-tdr-test#302

[2018-09-28 00:44] Validation started with lhcb-gaudi-head#2009

[2018-09-29 00:22] Validation started with lhcb-tdr-test#303

[2018-09-29 00:24] Validation started with lhcb-sanitizers#18

[2018-09-29 00:25] Validation started with lhcb-gaudi-head#2010

[2018-09-29 00:34] Validation started with lhcb-lcg-dev3#663

[2018-09-29 00:36] Validation started with lhcb-dd4hep#11

[2018-09-29 08:40] Validation started with lhcb-gaudi-head#2011

[2018-09-29 08:48] Validation started with lhcb-tdr-test#304

[2018-09-30 00:08] Validation started with lhcb-dd4hep#12

[2018-09-30 00:08] Validation started with lhcb-gaudi-head#2012

[2018-09-30 00:11] Validation started with lhcb-sanitizers#19

[2018-10-01 00:09] Validation started with lhcb-lcg-dev3#665

[2018-10-01 00:10] Validation started with lhcb-lcg-dev4#669

[2018-10-01 00:12] Validation started with lhcb-dd4hep#13

[2018-10-01 00:13] Validation started with lhcb-gaudi-head#2013

[2018-10-01 10:14] Validation started with lhcb-gaudi-head#2014

[2018-10-02 00:09] Validation started with lhcb-lcg-dev3#666

[2018-10-02 00:10] Validation started with lhcb-tdr-test#307

[2018-10-02 00:11] Validation started with lhcb-lcg-dev4#670

[2018-10-02 00:12] Validation started with lhcb-sanitizers#21

[2018-10-02 00:16] Validation started with lhcb-gaudi-head#2015

[2018-10-03 00:07] Validation started with lhcb-lcg-dev4#671

[2018-10-03 00:08] Validation started with lhcb-lcg-dev3#667

[2018-10-03 00:10] Validation started with lhcb-tdr-test#308

[2018-10-03 00:11] Validation started with lhcb-dd4hep#15

[2018-10-03 00:15] Validation started with lhcb-gaudi-head#2016

[2018-10-03 00:26] Validation started with lhcb-sanitizers#22

[2018-10-04 00:08] Validation started with lhcb-dd4hep#16

[2018-10-04 00:11] Validation started with lhcb-lcg-dev4#672

[2018-10-04 00:12] Validation started with lhcb-tdr-test#309

[2018-10-04 00:18] Validation started with lhcb-gaudi-head#2017

[2018-10-04 08:36] Validation started with lhcb-tdr-test#310

[2018-10-05 00:04] Validation started with lhcb-lcg-dev3#669

[2018-10-05 00:04] Validation started with lhcb-lcg-dev4#673

[2018-10-05 00:05] Validation started with lhcb-dd4hep#17

[2018-10-05 00:08] Validation started with lhcb-tdr-test#311

[2018-10-05 00:09] Validation started with lhcb-gaudi-head#2018

[2018-10-05 00:15] Validation started with lhcb-sanitizers#24

[2018-10-05 14:08] Validation started with lhcb-dd4hep#18

[2018-10-06 00:09] Validation started with lhcb-lcg-dev4#674

[2018-10-06 00:09] Validation started with lhcb-lcg-dev3#670

[2018-10-06 00:10] Validation started with lhcb-dd4hep#19

[2018-10-06 00:11] Validation started with lhcb-gaudi-head#2019

[2018-10-06 00:16] Validation started with lhcb-sanitizers#25

[2018-10-06 00:18] Validation started with lhcb-tdr-test#312

[2018-10-06 10:29] Validation started with lhcb-tdr-test#313

[2018-10-06 10:33] Validation started with lhcb-gaudi-head#2020

[2018-10-07 00:05] Validation started with lhcb-dd4hep#20

[2018-10-07 00:07] Validation started with lhcb-sanitizers#26

[2018-10-07 00:08] Validation started with lhcb-gaudi-head#2021

[2018-10-07 00:08] Validation started with lhcb-lcg-dev4#675

[2018-10-07 00:08] Validation started with lhcb-lcg-dev3#671

[2018-10-07 00:09] Validation started with lhcb-tdr-test#314

[2018-10-08 00:05] Validation started with lhcb-lcg-dev3#672

[2018-10-08 00:05] Validation started with lhcb-lcg-dev4#676

[2018-10-08 00:06] Validation started with lhcb-tdr-test#315

[2018-10-08 00:06] Validation started with lhcb-dd4hep#21

[2018-10-08 00:08] Validation started with lhcb-sanitizers#27

[2018-10-08 00:09] Validation started with lhcb-gaudi-head#2022

[2018-10-09 00:04] Validation started with lhcb-lcg-dev3#673

[2018-10-09 00:06] Validation started with lhcb-lcg-dev4#677

[2018-10-09 00:06] Validation started with lhcb-dd4hep#22

[2018-10-09 00:07] Validation started with lhcb-sanitizers#28

[2018-10-09 00:09] Validation started with lhcb-gaudi-head#2023

[2018-10-09 00:09] Validation started with lhcb-tdr-test#316

[2018-10-09 09:53] Validation started with lhcb-tdr-test#317

[2018-10-10 00:04] Validation started with lhcb-lcg-dev4#678

[2018-10-10 00:06] Validation started with lhcb-dd4hep#23

[2018-10-10 00:06] Validation started with lhcb-lcg-dev3#674

[2018-10-10 00:08] Validation started with lhcb-sanitizers#29

[2018-10-10 00:10] Validation started with lhcb-tdr-test#318

[2018-10-10 00:11] Validation started with lhcb-gaudi-head#2024

[2018-10-11 00:03] Validation started with lhcb-lcg-dev4#679

[2018-10-11 00:04] Validation started with lhcb-lcg-dev3#675

[2018-10-11 00:05] Validation started with lhcb-dd4hep#24

[2018-10-11 00:07] Validation started with lhcb-tdr-test#319

[2018-10-11 00:08] Validation started with lhcb-gaudi-head#2025

[2018-10-11 00:10] Validation started with lhcb-sanitizers#30

[2018-10-12 00:03] Validation started with lhcb-lcg-dev4#680

[2018-10-12 00:04] Validation started with lhcb-lcg-dev3#676

[2018-10-12 00:05] Validation started with lhcb-dd4hep#25

[2018-10-12 00:08] Validation started with lhcb-sanitizers#31

[2018-10-12 00:08] Validation started with lhcb-tdr-test#320

[2018-10-12 00:19] Validation started with lhcb-gaudi-head#2026

[2018-10-13 00:04] Validation started with lhcb-lcg-dev4#681

[2018-10-13 00:06] Validation started with lhcb-lcg-dev3#677

[2018-10-13 00:07] Validation started with lhcb-sanitizers#32

[2018-10-13 00:08] Validation started with lhcb-dd4hep#26

[2018-10-13 00:09] Validation started with lhcb-gaudi-head#2027

[2018-10-13 00:11] Validation started with lhcb-tdr-test#321

[2018-10-13 01:51] Validation started with lhcb-lcg-dev3#677

[2018-10-14 00:04] Validation started with lhcb-dd4hep#27

[2018-10-14 00:07] Validation started with lhcb-sanitizers#33

[2018-10-14 00:07] Validation started with lhcb-lcg-dev3#678

[2018-10-14 00:07] Validation started with lhcb-tdr-test#322

[2018-10-14 00:07] Validation started with lhcb-lcg-dev4#682

[2018-10-15 00:04] Validation started with lhcb-lcg-dev3#679

[2018-10-15 00:06] Validation started with lhcb-lcg-dev4#683

[2018-10-15 00:07] Validation started with lhcb-gaudi-head#2029

[2018-10-15 00:07] Validation started with lhcb-tdr-test#323

[2018-10-15 00:08] Validation started with lhcb-sanitizers#34

[2018-10-15 00:09] Validation started with lhcb-dd4hep#28

[2018-10-16 00:04] Validation started with lhcb-lcg-dev3#680

[2018-10-16 00:06] Validation started with lhcb-tdr-test#324

[2018-10-16 00:06] Validation started with lhcb-lcg-dev4#684

[2018-10-16 00:08] Validation started with lhcb-dd4hep#29

[2018-10-16 00:10] Validation started with lhcb-gaudi-head#2030

[2018-10-16 00:10] Validation started with lhcb-sanitizers#35

[2018-10-17 00:05] Validation started with lhcb-dd4hep#30

[2018-10-17 00:05] Validation started with lhcb-lcg-dev3#681

[2018-10-17 00:08] Validation started with lhcb-lcg-dev4#685

[2018-10-17 00:08] Validation started with lhcb-tdr-test#325

[2018-10-17 00:09] Validation started with lhcb-sanitizers#36

[2018-10-17 00:11] Validation started with lhcb-gaudi-head#2031

[2018-10-18 00:04] Validation started with lhcb-lcg-dev4#686

[2018-10-18 00:05] Validation started with lhcb-dd4hep#31

[2018-10-18 00:07] Validation started with lhcb-lcg-dev3#682

[2018-10-18 00:07] Validation started with lhcb-tdr-test#326

[2018-10-18 00:10] Validation started with lhcb-sanitizers#37

[2018-10-18 00:10] Validation started with lhcb-gaudi-head#2032

[2018-10-19 00:05] Validation started with lhcb-lcg-dev3#683

[2018-10-19 00:09] Validation started with lhcb-lcg-dev4#687

[2018-10-19 00:10] Validation started with lhcb-dd4hep#32

[2018-10-19 00:10] Validation started with lhcb-tdr-test#327

[2018-10-19 00:16] Validation started with lhcb-sanitizers#38

[2018-10-19 00:20] Validation started with lhcb-gaudi-head#2033

[2018-10-20 00:04] Validation started with lhcb-lcg-dev4#688

[2018-10-20 00:07] Validation started with lhcb-lcg-dev3#684

[2018-10-20 00:07] Validation started with lhcb-gaudi-head#2034

[2018-10-20 00:10] Validation started with lhcb-dd4hep#33

[2018-10-20 00:10] Validation started with lhcb-sanitizers#39

[2018-10-21 00:04] Validation started with lhcb-dd4hep#34

[2018-10-21 00:05] Validation started with lhcb-gaudi-head#2035

[2018-10-21 00:05] Validation started with lhcb-lcg-dev4#689

[2018-10-21 00:07] Validation started with lhcb-lcg-dev3#685

[2018-10-21 00:08] Validation started with lhcb-tdr-test#329

[2018-10-21 00:08] Validation started with lhcb-sanitizers#40

[2018-10-22 00:04] Validation started with lhcb-lcg-dev4#690

[2018-10-22 00:04] Validation started with lhcb-lcg-dev3#686

[2018-10-22 00:05] Validation started with lhcb-dd4hep#35

[2018-10-22 00:09] Validation started with lhcb-gaudi-head#2036

[2018-10-22 00:09] Validation started with lhcb-tdr-test#330

[2018-10-22 00:12] Validation started with lhcb-sanitizers#41

Edited 6 years ago by Software for LHCb
Marco Clemencic added 1 commit 6 years ago
added 1 commit

5a2eeb6a - Reorganization of Gaudi::Parsers code

Compare with previous version
Florian Lemaitre @flemaitr started a thread on an old version of the diff 6 years ago

Resolved 6 years ago by Marco Clemencic
Last reply by Marco Clemencic 6 years ago

Marco Clemencic added 2 commits 6 years ago

added 2 commits

45d98cd1 - Enable ADL for Gaudi::Parsers::parse
400d8ca3 - Reorganization of Gaudi::Parsers code

Compare with previous version

Marco Clemencic resolved all discussions 6 years ago

resolved all discussions

Marco Clemencic mentioned in merge request !761 (merged) 6 years ago

mentioned in merge request !761 (merged)

Marco Clemencic changed the description 6 years ago

changed the description

Marco Clemencic unmarked as a Work In Progress 6 years ago

unmarked as a Work In Progress

Marco Clemencic added 10 commits 6 years ago

added 10 commits

400d8ca3...22adb229 - 8 commits from branch gaudi:master
9bfa2615 - Enable ADL for Gaudi::Parsers::parse
2e5f39b5 - Reorganization of Gaudi::Parsers code

Compare with previous version

Marco Clemencic added label 6 years ago

added C++ framework label

Marco Clemencic mentioned in merge request !577 (merged) 6 years ago

mentioned in merge request !577 (merged)

Marco Clemencic assigned to @clemenci 6 years ago

assigned to @clemenci

Marco Clemencic assigned to @leggett 6 years ago

assigned to @leggett

Marco Clemencic added 16 commits 6 years ago

added 16 commits

2e5f39b5...22ddf725 - 14 commits from branch gaudi:master
fbb558c9 - Enable ADL for Gaudi::Parsers::parse
5e6de3fa - Reorganization of Gaudi::Parsers code

Compare with previous version

Marco Clemencic added 24 commits 6 years ago

added 24 commits

5e6de3fa...b010f5ef - 22 commits from branch gaudi:master
3bd4cd0f - Enable ADL for Gaudi::Parsers::parse
639ebd7e - Reorganization of Gaudi::Parsers code

Compare with previous version

Please register or sign in to reply