Re: [Wireshark-commits] rev 38515: /trunk/gtk/ /trunk/gtk/: capture_dlg.c capture_if_dlg.c

[Michael Tüxen <Michael.Tuexen () lurchi franken de>]

On Aug 14, 2011, at 2:04 AM, Guy Harris wrote:

> 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?
Probably. I'll put that on Irenes ToDo list...

Best regards
Michael
> ___________________________________________________________________________
> 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