On Thu, Mar 13, 2014 at 3:36 AM, Uros Bizjak <ubiz...@gmail.com> wrote: > > Attached patch changes the return value of the bzero macro to void, as > defined in a 4.3BSD: > > void bzero(void *s, size_t n); > > As an additional benefit, the changed macro now generates warning when > its return value is used (which is *not* the case in regex.c):
I'm not worried about anybody using the return value incorrectly in this file. I think we should just # define bzero(s, n) memset (s, '\0', n) I'll approve that change if it works. Ian