Sweeping !20930 from master to 21.3

Trivial change dropping unnecessary brackets, which give a warning in gcc8. Cherry-picking to 21.3 to keep the code in sync.

See merge request !20930 (merged)

(cherry picked from commit cb0dab2a)

41c113d2 TrigInDetAnalysisUser: Fix gcc8 compilation warning.

This merge request affects 1 package:

• Trigger/TrigAnalysis/TrigInDetAnalysisUser

Adding @sutt as watcher

added 21.3 Trigger review-pending-level-1 labels

CI Result FAILURE

	Athena
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-21063-2019-02-12-20-12
Athena: number of compilation errors 0, warnings 165
For experts only: Jenkins output [CI-MERGE-REQUEST 33690]

The q-test reference was already wrong earlier today, this test failure has nothing to do with this MR.

Thanks Rafal.

Cheers Mark

Regarding the triviality of this MR and that it is not related to the failing test at all, the MR can be approved.

Kira (L1)

added review-approved label and removed review-pending-level-1 label

This merge request affects 1 package:

• Trigger/TrigAnalysis/TrigInDetAnalysisUser

Adding @sutt as watcher

added review-pending-level-1 label and removed review-approved label

CI Result FAILURE

	Athena
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-21063-2019-02-13-15-15
Athena: number of compilation errors 0, warnings 166
For experts only: Jenkins output [CI-MERGE-REQUEST 33736]

The code in this request is not run as part of any athena required test - and indeed it is only used to build a standalone (non-athena) executable that we use to make plots for our validation.

Therefore, since the code builds without issue, there is no impediment to having it merged directly, so many thanks you to Rafal and Kira for looking at this.

Cheers Mark

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 9c64e5df

Many thanks John.

Cheers Mark

This merge request affects 1 package:

• Trigger/TrigAnalysis/TrigInDetAnalysisUser

Adding @sutt as watcher

added review-pending-level-1 label and removed review-approved label

CI Result FAILURE

	Athena
externals
cmake
make
required tests
optional tests

Full details available at NICOS MR-21063-2019-02-13-22-56
Athena: number of compilation errors 0, warnings 166
For experts only: Jenkins output [CI-MERGE-REQUEST 33767]

added sweep:done label

mentioned in commit c1da7acd

mentioned in merge request !21124 (merged)

Sweep summary
successful:

• 21.9

(see https://aiatlas154.cern.ch/jenkins/job/auto_git_sweeps/679/console for details)

added review-approved label and removed review-pending-level-1 label

mentioned in commit 417086d7