Simplify gaudisequencer

• re-use Gaudi::Details::ostream_joiner
• avoid heap allocation
• avoid recursive variadic pack expansion

• [2017-11-10 13:28] Validation started with lhcb-gaudi-merge#282
• [2017-11-11 00:04] Validation started with lhcb-future#598
• [2017-11-11 00:04] Validation started with lhcb-future-clang#372
• [2017-11-12 00:02] Validation started with lhcb-future-clang#373
• [2017-11-12 00:03] Validation started with lhcb-future#599
• [2017-11-13 00:03] Validation started with lhcb-future#600
• [2017-11-13 00:04] Validation started with lhcb-future-clang#374

mentioned in merge request !411 (merged)

note: this MR sits on top of !411 (merged).

@graven - can you rebase please?

assigned to @hegner

changed milestone to %v30r0

added 27 commits

• 7627a50e...8d59d401 - 25 commits from branch gaudi:master
• 4093ae70 - prefer C++14 ..._t<T> over typename ...<T>::type
• dbbcbffe - Simplify GaudiSequencer

Compare with previous version

Hi @graven,

I noticed a couple more lines in the tests(eg. 2012-bigevents) of the lhcb-future nightlies today.

The new lines are the following:

StatusCodeSvc        INFO listing all unchecked return codes:
StatusCodeSvc        INFO 
Num | Function                       | Source Library
----+--------------------------------+------------------------------------------
305 | GaudiSequencer::decodeNames()  | libGaudiAlgLib.so

Could this be caused by this MR?

added 1 commit

• c75c005b - Simplify GaudiSequencer

Compare with previous version

@chasse: thanks, fixed here

added 1 commit

• ea864633 - two temporary string copies less

Compare with previous version

added 1 commit

• 7d5bae8b - two temporary string copies less

Compare with previous version

mentioned in merge request lhcb/LHCb!941 (merged)

approved this merge request

merged

mentioned in commit aa8dc7b6

added C++ framework label