Re: ib - r30462 - abiword/trunk/src/af/ev/gtk2

From: Hub Figuière <hfiguiere_at_teaser.fr>
Date: Thu Nov 17 2011 - 16:49:07 CET

On 11-11-17 5:46 AM, cvs@abisource.com wrote:

> Modified: abiword/trunk/src/af/ev/gtk2/ev_UnixToolbar.cpp
> ===================================================================
> --- abiword/trunk/src/af/ev/gtk2/ev_UnixToolbar.cpp 2011-11-17 13:34:52 UTC (rev 30461)
> +++ abiword/trunk/src/af/ev/gtk2/ev_UnixToolbar.cpp 2011-11-17 13:46:40 UTC (rev 30462)
> @@ -1312,7 +1312,8 @@
> _wd * wd = m_vecToolbarWidgets.getNthItem(k);
> UT_ASSERT(wd && wd->m_widget);
> gtk_widget_set_sensitive(wd->m_widget, !bGrayed);
> - gtk_widget_set_visible(wd->m_widget, !EV_TIS_ShouldBeHidden(tis));
> + if (EV_TIS_ShouldBeHidden(tis)) gtk_widget_hide(wd->m_widget);
> + else gtk_widget_show(wd->m_widget);
> }
> break;
>
Please at lest make the code readable like
if ()
   statement;
else
   statement;

(or better, add the braces which I'm all for)

I know C++ is not Python but indentation ads clarity to the code.

Thanks,

Hub
Received on Thu Nov 17 16:49:31 2011

This archive was generated by hypermail 2.1.8 : Thu Nov 17 2011 - 16:49:31 CET