Re: Don't be fancy in stderr_out [Was Re: [FAILURE] CVS @ 10/05/2000 12:12]
Jan Vroonhof <vroonhof(a)math.ethz.ch> writes:
+#ifdef HAVE_VSNPRINTF
+ retval = vsnprintf ((char *) kludge, 8191, fmt, args);
+#else
retval = vsprintf ((char *) kludge, fmt, args);
- TO_EXTERNAL_FORMAT (DATA, (kludge, strlen ((char *) kludge)),
+#endif
...
+ kludge[8191] = 0;
Sorry for nitpicking, but could you please use `sizeof (kludge) - 1'
instead of 8191? For instance, I'd be happier if the last line were:
kludge[sizeof (kludge) - 1] = '\0';
You never know when someone will change the size of KLUDGE and miss an
instance of hardcoded 8191 or 8192.