Is there a reason why quick_add_integer_constant takes a char* and a length, instead of a (much quicker and cleaner) struct pike_string* which would enable it to work more efficiently in collaboration with MK_STRING() ?
I see that the functions do not seem to be used anywhere, so I propose to change the (internal) interface definition for them to actually make them quick.