Jaap Keuter <jaap.keu...@xs4all.nl>
>
>What I would like to see is the abolishment of the pwc_packet_properties_t type
>altogether. This is _not_ an enum.
I agree, and have come across another such "enum abuse" instance that may be
harder to address. Specifically, in the header file gmessages.h there is a
similar typedef enum called GLogLevelFlags which is also intended to provide
names for bitflags rather than actually enumerate. This construct is used a
number of places in the Wireshark code (such as line 3169 of
plugins/asn1/packet-asn1.c). Adding a cast quiets the compiler, but is there a
better way that doesn't require rewriting glib-2.0?
mylogh = g_log_set_handler (NULL, GLogLevelFlags(G_LOG_LEVEL_MASK |
G_LOG_FLAG_FATAL
| G_LOG_FLAG_RECURSION),
my_log_handler, NULL);
Ed
___________________________________________________________________________
Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org>
Archives: http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe
