http://linux.about.com/library/cmd/blcmdl3_va_start.htm va_end Each invocation of va_start must be matched by a corresponding invocation of va_end in the same function. 804 js_printf(JSPrinter *jp, const char *format, ...) 806 va_list ap; 813 va_start(ap, format); 816 if (*format == '\t') { 819 return -1; 826 if (!fp) 827 return -1; 838 if (!bp) { 840 return -1; 848 va_end(ap);

brendan gave me an rs for these changes (i'm recording that in this comment)

If you ever want to see it land, you may find that this once you'll need to land it yourself - jseng patches are supposed to land in tracemonkey first, and nobody who does checkin-needed has or seem to want to have a tracemonkey tree or the desire to land there, and judging by the patches that have been checkin-needed for six or seven months, nobody associated with jseng is going to step up and do them.