Hi Francois,
/* if we set the bufferpointer to NULL here UrlUnescape fails and string gets not converted */
/* if we set the buffer pointer to NULL here UrlUnescape fails and the string is not converted */
for consistency with the other buffer pointer comments and better readability, I suggest you add a comma here:
+ /* if we set the buffer pointer to NULL here, UrlUnescape fails and the string is not converted */
Best regards, Julian