a5a15b makes rb_fatal identical to Pike_fatal when debugging is not active. This doesn't work, because Pike_fatal takes just sprintf format and args, but rb_fatal takes a node pointer first. Since variadic macros exist only in ISO C 1999 and newer, I'm guessing they shouldn't be used; otherwise, it'd be simple to change the definition of rb_fatal to:
#define rb_fatal(NODE, ...) Pike_fatal(__VA_ARGS__)
What's the best way to go about this?
ChrisA