[Geany-devel] switch dialog bug - Re: [ANNOUNCE] gproject - yet another geany project plugin
nick.treleaven at xxxxx
Wed Jun 9 16:17:08 UTC 2010
On Wed, 9 Jun 2010 15:10:22 +0200
Jiří Techet <techet at 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.
More information about the Devel