ayatana-commits team mailing list archive

[Robert Collins <robertc@xxxxxxxxxxxxxxxxx>]

On Wed, 2009-11-25 at 03:35 +0000, Ted Gould wrote:
> 
> +       if (entry->image != NULL) {
> +               gtk_box_pack_start(GTK_BOX(hbox),
> GTK_WIDGET(entry->image), FALSE, FALSE, 0);
> +       }
> +       if (entry->label != NULL) {
> +               gtk_box_pack_start(GTK_BOX(hbox),
> GTK_WIDGET(entry->label), FALSE, FALSE, 0);
> +       } 

I'd be tempted to have a helper function for this idiom, though its
right on the lower bound of being useful for that.

Is this something that can be tested?

 review; approve


-- 
https://code.launchpad.net/~ted/indicator-applet/dynamic-allocation/+merge/15230
Your team ayatana-commits is subscribed to branch lp:indicator-applet.