EventContext: make constructor explicit
All threads resolved!
All threads resolved!
We do not want to implicitly convert an integer to an EventContext
as
this would almost certainly result in an invalid event context.
We just had a case in ATLAS where something like the following code compiled fine and only at runtime we got an invalid EventContext error:
void foo(const EventContext& ctx);
int context; // variable incidentally named context that has nothing to do with an `EventContext`
foo(context);
Merge request reports
Activity
Filter activity
assigned to @clemenci
added lhcb-gaudi-head label
changed milestone to %v36r13
- [2023-04-21 00:06] Validation started with lhcb-gaudi-head#3538
- [2023-04-21 00:14] Validation started with lhcb-run2-gaudi-head#499
- [2023-04-22 00:06] Validation started with lhcb-gaudi-head#3539
- [2023-04-23 00:04] Validation started with lhcb-gaudi-head#3540
- [2023-04-23 00:12] Validation started with lhcb-run2-gaudi-head#500
- [2023-04-24 00:05] Validation started with lhcb-gaudi-head#3541
- [2023-04-24 00:15] Validation started with lhcb-run2-gaudi-head#501
- [2023-04-25 00:04] Validation started with lhcb-gaudi-head#3542
- [2023-04-25 00:26] Validation started with lhcb-run2-gaudi-head#502
- [2023-04-26 00:04] Validation started with lhcb-gaudi-head#3543
- [2023-04-26 00:16] Validation started with lhcb-run2-gaudi-head#503
- [2023-04-27 00:05] Validation started with lhcb-gaudi-head#3544
- [2023-04-27 00:25] Validation started with lhcb-run2-gaudi-head#504
- [2023-04-28 00:05] Validation started with lhcb-gaudi-head#3545
- [2023-04-28 00:21] Validation started with lhcb-run2-gaudi-head#505
- [2023-04-29 01:26] Validation started with lhcb-gaudi-head#3546
- [2023-04-29 01:36] Validation started with lhcb-run2-gaudi-head#506
- [2023-04-30 00:04] Validation started with lhcb-gaudi-head#3547
- [2023-04-30 00:12] Validation started with lhcb-run2-gaudi-head#507
- [2023-05-01 00:04] Validation started with lhcb-gaudi-head#3548
- [2023-05-02 00:04] Validation started with lhcb-gaudi-head#3549
- [2023-05-02 00:11] Validation started with lhcb-run2-gaudi-head#508
- [2023-05-03 00:04] Validation started with lhcb-gaudi-head#3550
- [2023-05-03 00:13] Validation started with lhcb-run2-gaudi-head#509
- [2023-05-04 00:04] Validation started with lhcb-gaudi-head#3551
- [2023-05-04 00:18] Validation started with lhcb-run2-gaudi-head#510
- [2023-05-04 10:17] Validation started with lhcb-gaudi-head#3552
- [2023-05-05 00:04] Validation started with lhcb-gaudi-head#3553
- [2023-05-05 00:26] Validation started with lhcb-run2-gaudi-head#511
Edited by Software for LHCbmentioned in merge request atlas/athena!62527 (merged)
- Resolved by Marco Clemencic
removed lhcb-gaudi-head label
mentioned in merge request lhcb/Online!827 (merged)
added lhcb-gaudi-head label
mentioned in merge request atlas/athena!62510 (merged)
mentioned in commit 91812cb6
Please register or sign in to reply