Re: [Wireshark-commits] rev 46431: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-afs.c

[Guy Harris <guy () alum mit edu>]

On Dec 6, 2012, at 4:18 PM, Bill Meier <wmeier () newsguy com> wrote:

> On 12/6/2012 6:44 PM, Guy Harris wrote:
>
> > Hmm.  Maybe the test for "can I enable -Wshadow" should be "does the
> > compiler support it *and* does it complain about names in prototype
> > declarations shadowing variables?", not just "does the compiler
> > support it?"
>
> My thoughts, exactly, but left for another day ...

Checked in as rev 46475, but, for some unknown reason, the "does it complain about names in prototype declarations 
shadowing variables?" test says "yes" on the Ubuntu buildbot; I can't reproduce it on my Ubuntu 12.04 virtual machine.  
Presumably the test compile is failing for some other reason.
___________________________________________________________________________
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