Re: Is this patch to semcheck.c valid (handling FT_SYSTEM_ID) ?

[Alexis La Goutte <alexis.lagoutte () gmail com>]

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 () netscape net> 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 () googlemail com>
> To: Developer support list for Wireshark <wireshark-dev () wireshark org>
> 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 () wireshark org>
> Archives:    http://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
>              mailto:wireshark-dev-request () wireshark org?subject=unsubscribe <wireshark-dev-request () wireshark 
> org?subject=unsubscribe>
>
>
> ___________________________________________________________________________
> 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-request () wireshark org
> ?subject=unsubscribe
___________________________________________________________________________
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-request () wireshark org?subject=unsubscribe