On Mon, 2005-06-06 at 17:38, Robert Story wrote:
> On Mon, 06 Jun 2005 17:02:42 +0100 Dave wrote:
> DS> What would actually break if the declaration was changed [to]
> DS>      netsnmp_handler_args *cache_hint;
> DS> 
> DS> If anything is currently using this field, they must be casting
> DS> it to a 'netsnmp_handler_args*' anyway.  That cast would become
> DS> redundant, but I don't think it would actually break.


>  Making it the default, however, could, in theory, break the
> almost-certainly non-existent user who was using the pointer for
> some other type of hinting...

But the 'cache_hint' field is documented as being set by the cache
helper.  And the only use that the helper makes of this is to
provide the netsnmp_handler_args information.

Anyone using the netsnmp_cache mechanism *outside* of the
cache helper (such as the "extend" handling) should be using
the 'magic' field instead.

I propose that we make this change, and see how it turns out....

Dave



-------------------------------------------------------
This SF.Net email is sponsored by: NEC IT Guy Games.  How far can you shotput
a projector? How fast can you ride your desk chair down the office luge track?
If you want to score the big prize, get to know the little guy.  
Play to win an NEC 61" plasma display: http://www.necitguy.com/?r=20
_______________________________________________
Net-snmp-coders mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to