secp256k1_group_order_bytes constant in testutil.h. Noticed while reviewing the sending tests of #1698, which also adds another instance (called ORDERC there), i.e. including this it would be 5 instances in total (maybe there are even more around that I missed due to insufficient greping).
10@@ -11,6 +11,14 @@
11 #include "testrand.h"
12 #include "util.h"
13
14+/* group order of the secp256k1 curve in 32-byte big endian representation */
15+static const unsigned char secp256k1_group_order_bytes[32] = {
Fair question. It seems that we use capital letters mainly for constants and macros defined with the preprocessor (i.e. #define ...), where as actual global constant objects are still in lowercase, see e.g.
0$ git grep "^static const"
vs.
0$ git grep "^#define"
Maybe it’s worth it to add that to the style conventions section in CONTRIBUTING.md (and fix the few exceptions that don’t follow this conventions, e.g. SECP256K1_SIGNED{30,62}_ONE).
Don’t you feel this is one of those things so widely accepted that it’s strange to have to formally state it? It’s like seeing someone write class members in capital letters.
In any case, it was a tiny question anyway. The upside of this change is that we can script any future change, since everything uses the same variable now.
