Wireshark mailing list archives
Re: Still compile problems with capture_dlg.c
From: Michael Tuexen <Michael.Tuexen () lurchi franken de>
Date: Sat, 6 Aug 2011 19:39:57 +0200
On Aug 6, 2011, at 5:42 PM, Joerg Mayer wrote:
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=38375Thanks, 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 1I 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).
Hi Joerg, from what I could see is that the functions (I #if 0 out) are never called. I just wanted to keep the code if it turns out that it is needed when Irene is working on implementing the suggestions. I've put an item on our ToDo list to finally cleanup the source code. I'm wondering why our compilers did not complain... Best regards Michael
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
___________________________________________________________________________ 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:
- Still compile problems with capture_dlg.c Joerg Mayer (Aug 06)
- Re: Still compile problems with capture_dlg.c Michael Tuexen (Aug 06)
- Re: Still compile problems with capture_dlg.c Joerg Mayer (Aug 06)
- Re: Still compile problems with capture_dlg.c Michael Tuexen (Aug 06)
- Re: Still compile problems with capture_dlg.c Joerg Mayer (Aug 06)
- Re: Still compile problems with capture_dlg.c Michael Tuexen (Aug 06)
- Re: Still compile problems with capture_dlg.c Joerg Mayer (Aug 06)
- Re: Still compile problems with capture_dlg.c Michael Tuexen (Aug 06)