ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 4413] Improve bitview display

Date: Mon, 24 May 2010 06:44:59 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4413

--- Comment #22 from Jakub Zawadzki <darkjames@xxxxxxxxxxxxxxxx> 2010-05-24 06:44:58 PDT ---
Ups, I haven't notice that PI_* use the same flags...

Thanks for patch, I don't quite get why some PI_* values are like bitmasks, and
some not.

It might be good think to shrink it more:

-#define PI_RESPONSE_CODE    0x00040000
-#define PI_REQUEST_CODE    0x00050000
-#define PI_UNDECODED        0x00080000
-#define PI_REASSEMBLE        0x00100000
-#define PI_MALFORMED        0x00200000
...
-#define PI_SECURITY        0x01000000

+#define PI_RESPONSE_CODE    0x00030000 << 9
+#define PI_REQUEST_CODE    0x00040000 << 9
+#define PI_UNDECODED        0x00050000 << 9
+#define PI_REASSEMBLE        0x00060000 << 9
+#define PI_MALFORMED        0x00070000 << 9
....
+#define PI_SECURITY        0x000a0000 << 9

#define REP_LITTLE_ENDIAN       0x80000000 // intended to use in flags??
#define FI_LITTLE_ENDIAN        REP_LITTLE_ENDIAN // so lets use.
#define FI_BIG_ENDIAN           0x40000000


and epan/wslua/init.lua still needs a patch ;/

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