On 10/28/2009 07:29 AM, Jerry Quinn wrote:
+ length = strlen (name); + if (mark_private) + name_string = build_string (length + 1, buf); + else + name_string = build_string (length + 1, name);
These two calls shouldn't be using the same length. I think the +1 in the old code was unnecessary, so you can just remove it from the second call.
Jason