Wireshark · Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 38515: /trunk/gtk/ /trunk/gtk/: capture_dlg.c capture_if

Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 38515: /trunk/gtk/ /trunk/gtk/: capt

From: Guy Harris <guy@xxxxxxxxxxxx>

Date: Sat, 13 Aug 2011 17:04:59 -0700

On Aug 13, 2011, at 1:50 PM, Michael Tüxen wrote:

> On Aug 13, 2011, at 10:26 PM, Stig Bjørlykke wrote:
> 
>> BTW, why do we build the same string ("<b>%s</b>\n<span
>> size='small'>%s</span>") 4 different places?
> Two for the initial setup of the interface list, one might be able
> to reduce it to one.

insert_new_rows() and update_options_table() have similar code - could it be made common code?

Follow-Ups:
- Re: [Wireshark-dev] [Wireshark-commits] rev 38515: /trunk/gtk/ /trunk/gtk/: capture_dlg.c capture_if_dlg.c
  - From: Michael Tüxen

References:
- Re: [Wireshark-dev] [Wireshark-commits] rev 38515: /trunk/gtk/ /trunk/gtk/: capture_dlg.c capture_if_dlg.c
  - From: Stig Bjørlykke
- Re: [Wireshark-dev] [Wireshark-commits] rev 38515: /trunk/gtk/ /trunk/gtk/: capture_dlg.c capture_if_dlg.c
  - From: Michael Tüxen

Prev by Date: [Wireshark-dev] buildbot failure in Wireshark (development) on Visual-Studio-Code-Analysis
Next by Date: Re: [Wireshark-dev] Time shift patch causes compile error
Previous by thread: Re: [Wireshark-dev] [Wireshark-commits] rev 38515: /trunk/gtk/ /trunk/gtk/: capture_dlg.c capture_if_dlg.c
Next by thread: Re: [Wireshark-dev] [Wireshark-commits] rev 38515: /trunk/gtk/ /trunk/gtk/: capture_dlg.c capture_if_dlg.c
Index(es):
- Date
- Thread