Wireshark-bugs: [Wireshark-bugs] [Bug 6887] Patch for wireshark to support IEEE 1722.1 packets
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6887
--- Comment #5 from Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> 2012-03-01 12:17:49 PST ---
(In reply to comment #4)
> The pinfo = NULL assignments were there because when I first wrote the
> functions I wasn't sure if I would need pinfo someday but wasn't using it yet.
> They were just to keep the compiler from complaining about unused variables.
> They were at the end so if I did use them I wouldn't forget they were NULL and
> break things. I will remove them once the final spec is approved if they turn
> out to be unnecessary.
Ah, okay. The correct way to do that is to mark the parameter as unused by
appending _U_ to it. If I have time to review this again tonight I could make
that change if you don't beat me to it.
--
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.
