Jani Nikula <jani at nikula.org> writes: > On Thu, 05 Apr 2012 08:33:23 -0300, David Bremner <david at tethera.net> > wrote: >> >> > - if (len < 3) >> > + if (!isxdigit ((unsigned char) p[1]) || >> > + !isxdigit ((unsigned char) p[2])) >> >> What happens if there are not two characters after the escape? Is this >> relying on calling isxdigit on the null terminator? > > It is, and technically there's nothing wrong with that. Would you prefer > explicit checks for '\0' in the if condition, for clarity? Or a comment > about it?
I think a comment would do. Or the checks if you prefer.
