Ralph Corderoy wrote: > They don't need to be checked because they're only used in those cases > where truncation, but still NUL-terminated, is valid. Kind of like when > `%.42s' is used in a lexer error message in case the token is runaway, > or 'cut -c 42'. Ken's saying that some of them are like that, e.g. > feeding back unknown errors from outside. And the function's comment > would make their intended use case very clear.
ah, ok. let's make it nec'y (via Makefile settings) to cast to void in that situation, to avoid compiler warnings. >> i use asprintf() for this kind of thing. > > It's nice, but it might do the formatting work twice, and the return > value needs checking, not just for "out of memory" errors, the char** is > not guaranteed to be NULL on error with GNU, and that checking conflicts > with the "minimal call-site change" that's my aim. got it. -- P Vixie _______________________________________________ Nmh-workers mailing list Nmh-workers@nongnu.org https://lists.nongnu.org/mailman/listinfo/nmh-workers
