On Thu, 21 Jul 2005 14:44:29 +0100 Dave wrote: DS> On Thu, 2005-07-21 at 09:07 -0400, Robert Story wrote: DS> > DS> The first relates to the return value of the 'delete_row' DS> > DS> routine. This is defined as returning a 'netsnmp_index*', DS> DS> > Yep. And the return value is hard-coded to NULL. I can only guess DS> > that maybe originally it was a remove funtion that was re-tooled, DS> > and the prototype didn't get changed. It should be void. DS> DS> That's sort of what I suspected. DS> So the obvious followup question is: DS> Should we "fix" this, or just grin and bear it?
I'm ok with fix for this one. DS> > DS> Is there any reason why 'all_helpers.h' shouldn't include DS> > DS> this helper along with all of the others? DS> > DS> > Not that I can think of. DS> DS> OK - so how about we uncomment that #include, and wait for Wes DS> to cotton on? ok by me. DS> > This was a conscious decision. The prototypes for the hooks, obviously, DS> > must use the netsnmp_index, because the hooks are the same for all DS> > tables. DS> DS> (apart from 'row_copy' - why was that defined differently?) hmm... well, it isn't, in my main cvs tree.. musta missed it initially. DS> (You're even more warped than I am, at times!) LOL.. I'm going to take that as a compliment. DS> I'm trying to keep this particular section as short as possible. DS> It's the MfD discussion that looks set to expand dramatically..... DS> (But that's one for another day) Boy oh boy, *that* is going to be a fun one. I'd recommend that you start with what is in CVS, and recommend your users to use 5.3 or later. -- Robert Story; NET-SNMP Junkie Support: <http://www.net-snmp.org/> <irc://irc.freenode.net/#net-snmp> Archive: <http://sourceforge.net/mailarchive/forum.php?forum=net-snmp-coders> You are lost in a twisty maze of little standards, all different. ------------------------------------------------------- SF.Net email is sponsored by: Discover Easy Linux Migration Strategies from IBM. Find simple to follow Roadmaps, straightforward articles, informative Webcasts and more! Get everything you need to get up to speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click _______________________________________________ Net-snmp-coders mailing list Net-snmp-coders@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
