Constify C constructors and flags tables

[MisterDA]

There's little value to this considering the constructors tables are static, used very locally, but this serves nonetheless as an indication to the programmer that the tables are not to be modified.

[gasche]

Looks fine to me.

[xavierleroy]

Nice touch. Now these tables will go in the readonly segment, where they belong.

[xavierleroy]

Can you add a Changes entry? Then I'll merge.

[avsm]

There's also another opportunity for the same treatment for static int posix_signals[] in runtime/sys.c, in case you want to constify that too.

[MisterDA]

Done and done!
While grepping, I've also found this:

ocaml/bytecomp/bytelink.ml

Line 488 in 683d0ca

output_string outchan "static int caml_code[] = {\n";

maybe the generated caml_code could be made const too, if it's not self-modifying, or no changes are made to the array?

[xavierleroy]

maybe the generated caml_code could be made const too, if it's not self-modifying, or no changes are made to the array?

If memory serves, the bytecode can be modified before execution to replace opcodes by code pointers into the interpreter (the so-called indirect threaded code technique).