Recently D8596 had to redeclare DBKeys in the header file since it is also used in salvage.cpp
As previously discussed in D7020, this moves the constants into walletdb.h using string_views to make them eval at compile time
ninja all check check-functional
If you are going to do that, you need to add test for the serialization routines. I don't see what making these constexpr really benefit anything. The optimizer is likely able to do it anyways.
yea this ain't worth it