Hello -- Clang warns me that the zero on the second last line here, before the comma operator, is unused: #define ALLOCA(size) \ (REGEX_MALLOC_CHECK (), \ __temp_alloca_size__ = (size), \ __temp_alloca_size__ > MAX_ALLOCA_VS_C_ALLOCA ? \ xemacs_c_alloca (__temp_alloca_size__) : \ (need_to_check_c_alloca ? xemacs_c_alloca (0) : 0, \ alloca (__temp_alloca_size__))) There’s already a sequence point after evaluating need_to_check_c_alloca, so as far as I can tell it brings nothing and we can just remove it. Ben, if you’re around, was there some other motivation to it? Best, Aidan -- ‘Liston operated so fast that he once accidentally amputated an assistant’s fingers along with a patient’s leg, […] The patient and the assistant both died of sepsis, and a spectator reportedly died of shock, resulting in the only known procedure with a 300% mortality.’ (Atul Gawande, NEJM, 2012) _______________________________________________ XEmacs-Beta mailing list XEmacs-Beta(a)xemacs.org http://lists.xemacs.org/mailman/listinfo/xemacs-beta