On Wed, 9 Jun 2010 15:10:22 +0200
Jiří Techet <tec...@gmail.com> wrote:

> > BTW, why do you destroy the dialog instead of hiding it?
> 
> It's just that I can write
> 
> if (switch_dialog)
> 
> instead of
> 
> if (switch_dialog && GTK_WIDGET_VISIBLE(switch_dialog))
> 
> but I can change that if your preference is different.

When I wrote the code I assumed it would be faster to keep the dialog
than reconstruct it each time, but I didn't do any benchmarking.

Another general issue is that doing more than one change in a
'patch' makes it harder to review.

Regards,
Nick
_______________________________________________
Geany-devel mailing list
Geany-devel@uvena.de
http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel

Reply via email to