Wireshark mailing list archives

Re: Still compile problems with capture_dlg.c


From: Joerg Mayer <jmayer () loplof de>
Date: Sat, 6 Aug 2011 17:42:05 +0200

On Sat, Aug 06, 2011 at 02:42:00PM +0200, Michael Tuexen wrote:
No, not a bug in the compiler, a bug in the code. I checked in a fix in
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=38375

Thanks, it is gone now.

/home/jmayer/work/wireshark/svn/trunk/gtk/capture_dlg.c: At top level:
/home/jmayer/work/wireshark/svn/trunk/gtk/capture_dlg.c:527:1: error: ‘iftype_combo_box_add’ defined but not 
used
/home/jmayer/work/wireshark/svn/trunk/gtk/capture_dlg.c:600:1: error: ‘iftype_combo_box_new’ defined but not 
used
/home/jmayer/work/wireshark/svn/trunk/gtk/capture_dlg.c:629:1: error: ‘iftype_combo_is_separator’ defined but 
not used
make[2]: *** [gtk/CMakeFiles/ui.dir/capture_dlg.c.o] Error 1
I checked in a fix for these warnings in
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=38376

Please let me know if it now compiled on your configuration.

Of course not ;-) because now others are unused. If you think that removing that
code (well, #if 0'ing) is the right way I will do that for the code until it
compiles (will take 2-4 additional iterations, forgot how many but I did this
locally earlier and thought that removing so much code couldn't be intentional :)
I've committed revision 38379 which surrounds all the possibly unused code by
#ifdef HAVE_PCAP_REMOTE_NEVERTRUE/#endif (all the code was previously
#ifdef HAVE_PCAP_REMOTE/#endif).

Ciao
      Joerg

-- 
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

Current thread: