On Tue, 7 Jun 2005 09:08:15 +0200 (MEST) Gunnar wrote:
GL> cmu-snmp2.1.2/agent/snmp_vars.c
GL>
GL> 2128 #ifdef sunV3
GL> 2133 int Interface_Scan_Next(Index, Name, Retifnet)
GL> ...
GL> 2148 if (strcmp(saveName, "ip") == 0) {
GL> 2149 ifnetaddr = ifnet.if_next;
GL> 2150 continue;
GL>
GL> So, that test has been around for a while - 1996 "#ifdef sunV3" later
GL> updated to "#if defined(sunV3) || defined(linux)". An "educated guess"
GL> would be that this piece of code *might* have been needed in "sunV3"
GL> and that it was kept with "linux" since "it did no harm".
hrmm... I wonder what the issue is on sunV3.
GL> I plan to supply "diff -c" soon. Would you prefer that to:
GL>
GL> 1) Delete the old code completely.
GL>
GL> 2) Keep the old code inline but between
GL> #ifdef OLD_if_unit
GL> #endif /* OLD_if_unit */
I'd say 3) move it inside the "#ifndef linux".
Your other option would be to configure with --enable-mfd-rewrites to get the
new style implementation, which doesn't have this comparison...
--
NOTE: messages sent directly to me, instead of the lists, will be deleted
unless they are requests for paid consulting services.
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.
-------------------------------------------------------
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
