diff options
| author | John Thacker <johnthacker@gmail.com> | 2023-06-06 21:25:06 -0400 |
|---|---|---|
| committer | John Thacker <johnthacker@gmail.com> | 2023-06-08 11:05:42 +0000 |
| commit | bda350d8fdab87c4cb54358abb44a0efa6cc5968 (patch) | |
| tree | 10d7756441f34a85b3ae0bafd9c9828f06bf8b02 /epan/dissectors/packet-gsmtap.c | |
| parent | a82d5b56d10ff0c25141b3b6ff9f46a7a62c426b (diff) | |
Don't have every dissector include wtap.h
The header for frame_data can forward declare an incomplete type
for wtap_rec, since it only takes a pointer to it.
This prevents every dissector from automatically including
wiretap/wtap.h
Add wiretap/wtap.h to some dissectors that need it.
Remove it from some other dissectors that had the explicit include
but don't actually need it.
A few other dissectors actually need wsutil/inet_addr.h but were
getting that via wtap.h - include what they actually need.
This reduces the number of files that are recompiled when
wiretap/wtap.h is touched from ~2500 to ~800.
Note that most of the dissectors that still include wiretap/wtap.h
really only need to use a WTAP_ENCAP_ value, and most of the rest
just need a pseudoheader. Those could be moved into another wiretap
include to further reduce recompilation.
Related to #19127
Diffstat (limited to 'epan/dissectors/packet-gsmtap.c')
| -rw-r--r-- | epan/dissectors/packet-gsmtap.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/epan/dissectors/packet-gsmtap.c b/epan/dissectors/packet-gsmtap.c index dc0c19ccf4..377d0ecb26 100644 --- a/epan/dissectors/packet-gsmtap.c +++ b/epan/dissectors/packet-gsmtap.c @@ -36,6 +36,7 @@ #include <epan/packet.h> #include <epan/conversation.h> #include <epan/dissectors/packet-gsm_rlcmac.h> +#include <wiretap/wtap.h> #include "packet-gsmtap.h" #include "packet-lapdm.h" |