On Wed, Oct 20, 2010 at 6:27 PM, Wes Hardaker <
[email protected]> wrote:
>
> Regarding this change:
>
> > CHANGES: snmplib: Added functions snmp_register_callback2() and
> > netsnmp_register_callback2(). In contrast with the existing callback
> > registration functions, the two new functions allow to specify which
> > function to call instead of free() to deallocate the data passed to a
> > callback function. NULL is also valid, which means no deallocation
> > should be performed. Note: this change makes the
> > netsnmp_callback_clear_client_arg() superfluous.
>
> Wouldn't it be better to just introduce the new pointer into the
> structure and call it if set, or treat as before if it was NULL?
>
> That way you wouldn't need the register2 routines and wouldn't need new
> functions.
>
These functions have been removed again in r19566, but without reintroducing
the leak-at-exit.
Bart.
------------------------------------------------------------------------------
Nokia and AT&T present the 2010 Calling All Innovators-North America contest
Create new apps & games for the Nokia N8 for consumers in U.S. and Canada
$10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing
Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store
http://p.sf.net/sfu/nokia-dev2dev
_______________________________________________
Net-snmp-coders mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
