Re: table_data helpers

On Fri, 22 Apr 2005 16:29:44 +0100 Dave wrote:
DS> I'm just getting back up to speed with what's changed
DS> with the development code during my sabbatical, and noticed
DS> that you've backed out the container-based "table_data"
DS> helper, moving this into a separate helper.
DS>
DS> The CVS commit message mentions problems with backward
DS> compatibility - can you expand on this? Robert and I
DS> mentioned a potential difficulty to you regarding the
DS> disappearance of the linked list pointers - is this what
DS> you are referring to here? If not, then what?

I believe so. I seem to recall that it broke the snmpd.conf 'table' token.
Wes also pointed out that anyone wanting to look at the data itself would have
been using the linked list pointers, thus breaking backwards compatibility.

DS> I'd really like to replace the previous table_data helper
DS> completely, if we can manage to do so safely.

It's the old backwards compatibility bug biting us again. I'm comfortable with
providing wrapper functions to provide an emulation of the linked list
pointers. I don't think having to change

item = item->next;

to
item = next_item(item);

is to onerous. But I've always lost this type of argument in the past. ;-)


DS> I'm not convinced that marking it as "deprecated" will have much effect.

Not on existing user code. But if all internal examples and mib2c conf files
are updated to use the new version, it should fade over time.

DS> Otherwise, I might look at making the new name a little more
DS> distinctive, to more clearly distinguish it from the old.

I did give him a hard time about the name. I agree that the new one sucks.

--
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 email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Net-snmp-coders mailing list
Net-snmp-coders@lists.sourceforge.net
https://lists.sourceforge.net/lists/...et-snmp-coders