Jani Nikula <[email protected]> writes: > On Thu, 05 Apr 2012 08:33:23 -0300, David Bremner <[email protected]> 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. _______________________________________________ notmuch mailing list [email protected] http://notmuchmail.org/mailman/listinfo/notmuch
