Wireshark-bugs: [Wireshark-bugs] [Bug 7658] LLRP: Dissect parameter fields
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Sat, 8 Sep 2012 18:47:23 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7658

Evan Huus <eapache@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #9099|review_for_checkin?         |review_for_checkin+
              Flags|                            |

--- Comment #10 from Evan Huus <eapache@xxxxxxxxx> 2012-09-08 18:47:22 PDT ---
Comment on attachment 9099
  --> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=9099
Getting more traditional

Thanks for the patch, committed in revision 44817.

I expanded one set myself because they were variadic, which not all our
compilers support. Don't worry about the time - the macros were a code
readability issue, not a bug, and I don't think anyone else is trying to read
the dissector code at the moment :)

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.