On Fri, Apr 3, 2026 at 10:09 PM Lukas Fittl <[email protected]> wrote: > Maybe make it clear in the comment that the return value is > unterminated (e.g. "The return value is the unterminated identifier"), > i.e. caller is responsible for setting the null byte based on endp. > > Otherwise 0001 looks good.
Thanks for the review, but I don't agree with that proposed change. To me, it seems too much like insisting that a function must document the things it doesn't do. Granted, saying "this function doesn't overwrite *endp" is a much more reasonable thing to document than "this function doesn't try to compute pi to 3,141,593 decimal places," because the latter is less likely to be something that the user would expect it to do. But I feel like the comment is clear enough as it is, so I've committed the patch without change. If more votes in favor of adjusting the comment emerge, we can certainly reconsider. -- Robert Haas EDB: http://www.enterprisedb.com
