ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
July 17th, 2024 | 10:00am-11:55am SGT (UTC+8) | Online

Ethereal-dev: RE: [Ethereal-dev] Updates to COPS & SNMP dissectors

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

Date: Tue, 9 Sep 2003 10:37:14 +0300
Hi !

KTi> -----Original Message-----
KTi> From: ext Guy Harris [mailto:guy@xxxxxxxxxxxx]
KTi> Sent: 08 September, 2003 23:46
KTi> To: Tiirikainen Kari (NET-OSS/Espoo)
KTi> Cc: ethereal-dev@xxxxxxxxxxxx
KTi> Subject: Re: [Ethereal-dev] Updates to COPS & SNMP dissectors
KTi>
KTi> That patch didn't apply correctly, as it wasn't a "-c" or 
KTi> "-u" patch,  but, fortunately, it wasn't *too* wrong, so I could fix 
KTi> the damage by hand.

Thank you, although there wasn't such a rush,
so by now you should have gotten the correct diff.

KTi> However, I changed it so that there's a routine 
KTi> "process_prefs()" which 
KTi> is called when the preferences change; it unloads the MIBs 
KTi> if they've 
KTi> been loaded (false the first time through, true all 
KTi> subsequent times through), exports the preference setting,
KTi> and then loads the MIBs.

OK, sounds fine to me.

KTi> Note that there is no such thing as a preference that 
KTi> hasn't been set, 
KTi> i.e., by the time the preferences are read, it is 
KTi> *guaranteed* to be 
KTi> the case that "mib_modules" is set - and that'd be the 
KTi> case even if the 
KTi> register routine *didn't* give it an initial value.

OK, good to know. I checked packet-diameter.c for reference
when coding the preference setting and there was the checking too,
so I used it too.
 
KTi> That's what's currently checked in.

Looks great, thanks.

Regards,
Kari