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

Reply via email to