[PATCH 1/3] unify xmalloc prototypes & friends
Roland McGrath
roland@hack.frob.com
Wed Jan 23 23:53:00 GMT 2013
This is pretty well independent from the other two patches and if I were
you I would have just posted it separately. If it's part of the same
series I would have made it the last patch in the series.
I agree with the sentiments expressed and I think this is a good start.
But locale/programs/ is indeed wrong. Even if this is all you are doing
now, you can create include/programs/ and put a file in there. Since this
header just declares the x* memory functions, I'd prefer it have a less
generic name reflecting what's actually in it. (And if there are other
different declarations to put in a common header in the future, there's
no particular reason those ought to go in the same one rather than in a
different topical header in include/programs/.)
Thanks,
Roland
More information about the Libc-alpha
mailing list