Re: [Wireshark-commits] rev 43413: /trunk/ui/gtk/ /trunk/ui/gtk/: main_80211_toolbar.c

[Joerg Mayer <jmayer () loplof de>]

On Wed, Jun 20, 2012 at 05:35:38PM +0000, etxrab () wireshark org wrote:
> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=43413
>
> User: etxrab
> Date: 2012/06/20 10:35 AM
>
> Log:
>  #if 0 out code to make it build in it's current state
>
> Directory: /trunk/ui/gtk/
>   Changes    Path                    Action
>   +3 -2      main_80211_toolbar.c    Modified

This of course caused another build failure (with warnings as errors)
due to now unused parameters.
Wouldn't it make more sense to comment the code back in
and add the changes to capture_sync.[hc] instead, thus avoiding the
missing function.

ciao
 Jörg
-- 
Joerg Mayer                                           <jmayer () loplof de>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
___________________________________________________________________________
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