On Tue, Oct 28, 2008 at 10:32 PM, Alexey Melnikov <[EMAIL PROTECTED]> wrote:
> This looks reasonable. > > However, I am wondering if the following line: > > + memset(value, 0, sizeof(value)); > > can be replaced with: > > + value[0] = '\0'; > > Can you test? Tested. It works. > Also, please add this to bugzilla, if you haven't yet. That's done, but with initial version of the patch attached: http://bugzilla.andrew.cmu.edu/show_bug.cgi?id=3111