<div dir="ltr"><div>Here is the revised patch after cleaning up unnecessary bits of code.<br><br></div>Anton, can you please review this and tell me if the needed changes have been done.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 18, 2014 at 12:48 PM, Anton Lundin <span dir="ltr"><<a href="mailto:glance@acc.umu.se" target="_blank">glance@acc.umu.se</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 18 December, 2014 - Nikhil Bharadwaj wrote:<br>
<br>
</span><span class="">> Anton,<br>
><br>
> I was not sure if UserSurvey worked on Android. Thats the reason I added<br>
> it.<br>
<br>
</span>Please, don't top-post, and please, don't add such knobs if there not<br>
needed, and if they are needed they should be added everywhere.<br>
<span class=""><br>
> And yeah, I overlooked the #include <usersurvey.h> on line 54.<br>
><br>
<br>
</span>Now Dork tok the patch, but please, send a following commit cleaning<br>
this up.<br>
<br>
> Sorry for that.<br>
><br>
<br>
NP.<br>
<br>
//Anton<br>
<div class="HOEnZb"><div class="h5"><br>
> Nikhil Bharadwaj<br>
><br>
> On Thu, Dec 18, 2014 at 12:07 PM, Anton Lundin <<a href="mailto:glance@acc.umu.se">glance@acc.umu.se</a>> wrote:<br>
> ><br>
> > On 18 December, 2014 - Nikhil Bharadwaj wrote:<br>
> ><br>
> > > Resending the patch.<br>
> > ><br>
> > > On Wed, Dec 17, 2014 at 10:27 AM, Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> wrote:<br>
> > > ><br>
> > > > On Wed, Dec 17, 2014 at 10:19:21AM +0530, Nikhil Bharadwaj wrote:<br>
> > > > ><br>
> > > > > Okay, I'm sorry. I didnt know that. Okay Ill change it and send it<br>
> > to you<br>
> > > > > once you finish releasing 4.3<br>
> > > ><br>
> > > > Why would you be sorry? I'm thrilled that you are contributing and<br>
> > working<br>
> > > > on making the Subsurface user experience better. That is wonderful!<br>
> > > ><br>
> > > > I apologize that the timing is just unfortunate, otherwise I would of<br>
> > > > course have accepted your patches already...<br>
> > > ><br>
> > > > /D<br>
> > > ><br>
> > ><br>
> > ><br>
> > > --<br>
> > > Nikhil Bharadwaj<br>
> ><br>
> > > From be92f5c121520434e6919ec4e8382e4038a75205 Mon Sep 17 00:00:00 2001<br>
> > > From: nikhil <<a href="mailto:nikhil.gosala@gmail.com">nikhil.gosala@gmail.com</a>><br>
> > > Date: Wed, 17 Dec 2014 09:29:41 +0530<br>
> > > Subject: [PATCH 3/3] Added the option of opening User Survey form<br>
> > explicitly<br>
> > ><br>
> > > Earlier, there was no option for the user to explicitly open the User<br>
> > Survey<br>
> > > form. This has been corrected by placing an option in the "Help" menu by<br>
> > which<br>
> > > the user can explicity open the User Survey form.<br>
> > ><br>
> > > Signed-off-by: Nikhil Bharadwaj Gosala <<a href="mailto:nikhil.gosala@gmail.com">nikhil.gosala@gmail.com</a>><br>
> > > ---<br>
> > >  qt-ui/mainwindow.cpp | 26 ++++++++++++++++++++++++++<br>
> > >  qt-ui/mainwindow.h   |  1 +<br>
> > >  qt-ui/mainwindow.ui  |  8 +++++++-<br>
> > >  3 files changed, 34 insertions(+), 1 deletion(-)<br>
> > ><br>
> > > diff --git a/qt-ui/mainwindow.cpp b/qt-ui/mainwindow.cpp<br>
> > > index acf520d..1765d4c 100644<br>
> > > --- a/qt-ui/mainwindow.cpp<br>
> > > +++ b/qt-ui/mainwindow.cpp<br>
> > > @@ -55,6 +55,9 @@<br>
> > >  #ifndef NO_USERMANUAL<br>
> > >  #include "usermanual.h"<br>
> > >  #endif<br>
> > > +#ifndef NO_USERSURVEY<br>
> ><br>
> > Why a NO_USERSURVEY?<br>
> ><br>
> > NO_PRINTING and NO_USERMANUAL exists due to none of those components<br>
> > work on Android, but the usersurvey works and i can't see a reson why<br>
> > disabling bits of the usersurvey?<br>
> ><br>
> > > +#include "usersurvey.h"<br>
> ><br>
> > Its already included, right out of context for this patch, line 54 in<br>
> > qt-ui/mainwindow.cpp<br>
> ><br>
> > > +#endif<br>
> > >  #include <QNetworkProxy><br>
> > ><br>
> > >  MainWindow *MainWindow::m_Instance = NULL;<br>
> > > @@ -118,6 +121,10 @@ MainWindow::MainWindow() : QMainWindow(),<br>
> > >  #ifdef NO_USERMANUAL<br>
> > >       ui.menuHelp->removeAction(ui.actionUserManual);<br>
> > >  #endif<br>
> > > +#ifdef NO_USERSURVEY<br>
> > > +     ui.menuHelp->removeAction(ui.actionUserSurvey);<br>
> > > +#endif<br>
> > > +<br>
> > >  #ifdef NO_PRINTING<br>
> > >       ui.menuFile->removeAction(ui.actionPrint);<br>
> > >  #endif<br>
> > > @@ -745,6 +752,16 @@ void MainWindow::on_actionUserManual_triggered()<br>
> > >  #endif<br>
> > >  }<br>
> > ><br>
> > > +void MainWindow::on_actionUserSurvey_triggered()<br>
> > > +{<br>
> > > +#ifndef NO_USERSURVEY<br>
> > > +     if(!survey) {<br>
> > > +             survey = new UserSurvey();<br>
> ><br>
> > Missing a connection to the parent here.<br>
> ><br>
> ><br>
> > //Anton<br>
> ><br>
> ><br>
> > --<br>
> > Anton Lundin    +46702-161604<br>
> ><br>
><br>
><br>
> --<br>
> Nikhil Bharadwaj<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Anton Lundin    +46702-161604<br>
</font></span></blockquote></div><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr">Nikhil Bharadwaj</div></div>
</div>