On Feb 4, 2008, at 1:21 AM, Andy Lester wrote:
SymReg * r = _get_sym_typed(hsh, name, t);
if (r) {
free(name);
return r;
}
It's really saying "If I don't need a copy, then free what I assume
is a copy", rather than "If I need a copy,
While we're at it, how about the horror of delete_ins() being told
whether it should be freeing its argument?
--
Andy Lester => [EMAIL PROTECTED] => www.petdance.com => AIM:petdance
On Feb 4, 2008, at 12:44 AM, chromatic wrote:
SymReg * r = _get_sym_typed(hsh, name, t);
if (r) {
free(name);
return r;
}
It's really saying "If I don't need a copy, then free what I assume is
a copy", rather than "If I need a copy,
