Small code clean up
-
TestFunctors.h: drop unused template sequence-
std::integer_sequence<SIMDWrapper::InstructionSet, ...>is not used and it can be removed. Moreover, it is an abuse ofstd::integer_sequence, becauseSIMDWrapper::InstructionSetis not an integral type and gcc14 does not accept it.
-
-
MaterialLocatorBase.cpp: simpler implementation offor_each_adjacent_pair-
Using
std::mismatchis a clever abuse of an STL algorithm for something it was not meant to do (hence the special trick of swappting twice the order of arguments and the artificialreturn true), but it doesn't bring much benefit (it actually makes the code difficult to understand). See b625a107Unfortunately, gcc 14 warns that we are not using what
std::mismatchreturn, so I changed the implementation to use a simple loop that is somehow more understandable (thanks to a LLM).
-
-
SelectiveBremMatchAlg.cpp: use the return value ofstd::all_of(as requested by gcc 14)
Edited by Marco Clemencic