Skip to content

Commit ed7fe8c

Browse files
committed
keep clang happy even though I don't believe it's right
Clang complains that: ``` SOURCES/QualityControl/2660/0/Framework/test/testActor.cxx /Volumes/build/alice-ci-workdir/qualitycontrol-o2/sw/SOURCES/QualityControl/2660/0/Framework/test/testActor.cxx:97:5: error: call to consteval function 'o2::quality_control::core::UnrequestedAccessActor::assertNoAccessToServices<o2::quality_control::core::UnrequestedAccessActor>' is not a constant expression 97 | assertNoAccessToServices<UnrequestedAccessActor>(); | ^ /Volumes/build/alice-ci-workdir/qualitycontrol-o2/sw/SOURCES/QualityControl/2660/0/Framework/test/testActor.cxx:97:5: note: implicit use of 'this' pointer is only allowed within the evaluation of a call to a 'constexpr' member function 1 error generated. ``` GCC is fine and is able to process it as a consteval expression. Let's keep both happy by using constexpr, given it's just a test.
1 parent c3f04c2 commit ed7fe8c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Framework/test/testActor.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ class UnrequestedAccessActor : public Actor<UnrequestedAccessActor>
8484
explicit UnrequestedAccessActor(const ServicesConfig& cfg) : Actor<UnrequestedAccessActor>(cfg) {}
8585

8686
template <typename T>
87-
consteval void assertNoAccessToServices()
87+
constexpr void assertNoAccessToServices()
8888
{
8989
static_assert(!(requires(T& t) { t.getMonitoring(); }));
9090
static_assert(!(requires(T& t) { t.getBookkeeping(); }));

0 commit comments

Comments
 (0)