Wireshark · Wireshark-dev: Re: [Wireshark-dev] Adding tap to dissector causes build errors in tap.h?

[Shawn Mayer <mayer_sr@xxxxxxxx>]

Please excuse my continued stupidity. What do you mean by grepping? 
Thank you for the solution, it builds just fine. Now to make more problems!

Shawn

On 4/11/2010 8:34 PM, Bill Meier wrote:
> Shawn Mayer wrote:
>    
> > After reverting back to the original file I found that tap.h throws
> > errors even if the only change I make is adding the "#include<tap.h>".
> > I've tried including it in packet-aim.c and packet-aim-messaging.c with
> > the same results:
> >
> >
> > register.c
> > packet-aim-messaging.c
> > c:\wireshark\epan\tap.h(31) : error C2143: syntax error : missing ')'
> > before '*'
> >
> > c:\wireshark\epan\tap.h(31) : error C2081: 'packet_info' : name in
> > formal parame
> > ter list illegal
> >      
> Looking at one of lines the compiler is complaining about:
>
> typedef gboolean (*tap_packet_cb)(void *tapdata, packet_info *pinfo,
> epan_dissect_t *edt, const void *data);
>
> it seems that the references to 'packet_info' and 'epan_dissect_t'  are
> probably the cause of the complaints.
>
> Doing a grep for these symbols over the wireshark .h source shows that
> these are defined in
>
> epan/packet_info.h
> and epan/epan.h
>
>
> Alternatively (and probably better): grepping the epan/dissector sources
> for dissector .c files which use tap.h  and then examining same, it
> becomes clear that
>
> including packet.h before including tap.h also works (since packet.h
> includes both packet_info.h and epan.h).
>
> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list<wireshark-dev@xxxxxxxxxxxxx>
> Archives:    http://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
>               mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe
>
>    

NTMail K12 - the Mail Server for Education