<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 9, 2015 at 5:09 PM, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Mar 09, 2015 at 04:01:12PM -0300, Tomaz Canabrava wrote:<br>
> From 7e89b79afbefd508015841a667cc68a73865754b Mon Sep 17 00:00:00 2001<br>
> From: Tomaz Canabrava <<a href="mailto:tomaz.canabrava@intel.com">tomaz.canabrava@intel.com</a>><br>
> Date: Mon, 9 Mar 2015 15:12:48 -0300<br>
> Subject: [PATCH 3/6] Remove redundant code<br>
><br>
> Another bit of piece of software that shouldn't exist.<br>
><br>
> Signed-off-by: Tomaz Canabrava <<a href="mailto:tomaz.canabrava@intel.com">tomaz.canabrava@intel.com</a>><br>
> ---<br>
>  qt-ui/maintab.cpp | 6 ++----<br>
>  1 file changed, 2 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/qt-ui/maintab.cpp b/qt-ui/maintab.cpp<br>
> index 956a1bc..dff49d8 100644<br>
> --- a/qt-ui/maintab.cpp<br>
> +++ b/qt-ui/maintab.cpp<br>
> @@ -71,6 +71,7 @@ MainTab::MainTab(QWidget *parent) : QTabWidget(parent),<br>
>       // filled from a dive, they are made writeable<br>
>       setEnabled(false);<br>
><br>
> +<br>
<br>
Really?<br></blockquote><div><br></div><div>Sorry, slipped in.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> @@ -955,10 +956,7 @@ void MainTab::markChangedWidget(QWidget *w)<br>
>       qApp->palette().color(QPalette::Text).getHslF(&h, &s, &l, &a);<br>
>       p.setBrush(QPalette::Base, (l <= 0.3) ? QColor(Qt::yellow).lighter() : (l <= 0.6) ? QColor(Qt::yellow).light() : /* else */ QColor(Qt::yellow).darker(300));<br>
>       w->setPalette(p);<br>
> -     if (!modified) {<br>
> -             modified = true;<br>
> -             enableEdition();<br>
> -     }<br>
> +     modified = true;<br>
<br>
Are you sure this is correct? I thought the first time something changes<br>
we do have to call enableEdition() - or is this taken care of somewhere<br>
else?<br></blockquote><div><br></div><div>in about two lines above :)<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
/D<br>
<br>
</font></span></blockquote></div><br></div></div>