Wireshark · Wireshark-dev: Re: [Wireshark-dev] Is this patch to semcheck.c valid (handling FT_SYSTEM_ID) ?

[Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>]

There is also : ./wslua_proto.c:1261:1: error: 'ProtoField_systemid' defined but not used [-Werror=unused-function]

(Fixed in revision 54397)

On Mon, Dec 23, 2013 at 6:59 PM, <mmann78@xxxxxxxxxxxx> wrote:

I believe I just missed committing that file when I added the FT_SYSTEM_ID type.  Committed (functionality) to rev 54394 and thanks for the catch!

 

-----Original Message-----
From: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
To: Developer support list for Wireshark <wireshark-dev@xxxxxxxxxxxxx>
Sent: Mon, Dec 23, 2013 12:44 pm
Subject: [Wireshark-dev] Is this patch to semcheck.c valid (handling FT_SYSTEM_ID) ?

My gcc build is complaining that FT_SYSTEM_ID is missing from 3 switch statements.  I'm not sure I properly understand how it is used, so would appreciate if someone could take a quick look at this patch?  If there are no objections I will commit it later...

Thanks,

Martin

___________________________________________________________________________
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

___________________________________________________________________________
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