Removed use of deprecated factories

• replaced DECLARE_*_FACTORY with DECLARE_COMPONENT
• made constructors public
• removed some unnecessary disabling of copy constructors
• removed some trivial destructors

See gaudi/Gaudi!420 (merged)

added 3 commits

• 7ce219b4...1845aa07 - 2 commits from branch master
• 5ec6474f - Removed use of deprecated factories

Compare with previous version

added 1 commit

• b3bddbb5 - Removed use of deprecated factories

Compare with previous version

added 4 commits

• b3bddbb5...1845aa07 - 3 commits from branch master
• 5ec6474f - Removed use of deprecated factories

Compare with previous version

• [2017-12-01 18:12] Validation started with prepare-gaudi-420#9
• [2017-12-02 09:53] Validation started with prepare-gaudi-420#9
• [2017-12-04 10:43] Validation started with prepare-gaudi-420#10
• [2018-02-20 00:22] Validation started with lhcb-gaudi-head#1775
• [2018-02-21 00:21] Validation started with lhcb-gaudi-head#1776
• [2018-02-22 01:21] Validation started with lhcb-gaudi-head#1777
• [2018-02-23 01:21] Validation started with lhcb-gaudi-head#1778
• [2018-02-24 01:21] Validation started with lhcb-gaudi-head#1779
• [2018-02-25 01:19] Validation started with lhcb-gaudi-head#1780
• [2018-02-26 01:26] Validation started with lhcb-gaudi-head#1781
• [2018-02-27 01:24] Validation started with lhcb-gaudi-head#1782
• [2018-02-28 01:24] Automatic merge failed in lhcb-gaudi-head#1783
• [2018-02-28 12:53] Validation started with lhcb-gaudi-merge#432
• [2018-02-28 12:59] Validation started with lhcb-gaudi-merge#433
• [2018-02-28 13:28] Validation started with lhcb-gaudi-merge#434
• [2018-02-28 13:31] Validation started with lhcb-gaudi-merge#435
• [2018-02-28 15:49] Validation started with lhcb-gaudi-merge#436
• [2018-03-01 00:16] Validation started with lhcb-gaudi-head-py3#79
• [2018-03-01 00:18] Validation started with lhcb-clang-test#876
• [2018-03-01 00:20] Validation started with lhcb-tdr-test#74
• [2018-03-01 00:20] Validation started with lhcb-head#1772
• [2018-03-01 00:28] Validation started with lhcb-gaudi-head#1784
• [2018-03-01 13:18] Validation started with lhcb-gaudi-merge#437

@clemenci You might want to rebase this on 2018-patches (also to fix the merge conflicts)?

added 61 commits

• 5ec6474f...b8a5ac4b - 60 commits from branch master
• b31960e5 - Removed use of deprecated factories

Compare with previous version

I rebased it on master, since it's not going to be needed for Gaudi v29. Moreover, some of the conflicts were due to !246 (merged).

@erodrigu FYI. I'm testing it in lhcb-gaudi-head tonight.

Thanks. There is also the related MR Phys!224 (merged) to update.

It's a bit of a shame this was not merged last week. It needs again to be rebased due to new merge conflicts....

Agreed. But this MR relates to gaudi/Gaudi!420 (merged), which is still marked as WIP. I am waiting for news from there ... BTW, that's the case for various MRs that have been marked as WIP or waiting for further info for a loonnng while. I'm chasing but do not always get replies, unfortunately.

This MR has to be applied ahead of gaudi/Gaudi!420 (merged), it contains the forward/backward compatibility fixes that will break compilation when gaudi/Gaudi!420 (merged) is applied!

Then this should have been made clear! Also, and more importantly, why is it still WIP? I cannot guess ... Removing the WIP ... Guess the same can be said of the same kind of MRs in Analysis and DaVinci, then...

unmarked as a Work In Progress

changed the description

mentioned in merge request Phys!224 (merged)

added 11 commits

• b31960e5...cb6c2543 - 10 commits from branch master
• 51e1329a - Removed use of deprecated factories

Compare with previous version

Rebased and fixed the conflicts.

merged

mentioned in commit 466f244a

mentioned in merge request !282 (closed)

mentioned in merge request !299 (merged)

mentioned in commit b0d94411

mentioned in merge request !500 (merged)