<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug 27, 2016 at 4:36 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"><span class="">On Sat, Aug 27, 2016 at 08:58:08AM -0300, Tomaz Canabrava wrote:<br>
> sorry for the long delay - life is a bit crazy.<br>
><br>
> For those who don't know yet, I'm trying to move to germany and I just went<br>
> to the consulate here in brazil this tuesday - send me good vibes.<br>
<br>
</span>How did it go? I assume they beat you up, threw you on the ground, used<br>
some tear gas on you and then threatened to sue you over the damage you<br>
did to their carpet when throwing up?<br></blockquote><div><br></div><div>It went fairly well - since I had *tons* of documentation to give to them, I had to do in turns because it didn't fit into the documentation-exchange mechanism (they have a small rotating glass where you put your documents) - I had to use it three times to pass all of my paperwork. <br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> While this is mostly finished and a *bit* tested, I didn't do a ton of<br>
> testing, so I'll send more patches over those, but this is a goos start, if<br>
> I break anything on any preferences - you'd been warned, not finished, but<br>
> mergeable.<br>
<br>
</span>It applies nicely. I did my usual rewriting of the commit messages and a<br>
couple of tiny whitespace fixes.<br>
<span class=""><br>
> I'd say this is 80% ready. I'v foud a few places where a setting is changed<br>
> the 'wrong' way (and by wrong way I mean that a setting is changed, the<br>
> setting should trigger code but some unrelated signal is send that in turn<br>
> will trigger the correct code. But the signal (or the class) shouldn't be<br>
> doing that)<br>
<br>
</span>I compiled it and started testing. First thing I notice, I get asked if I<br>
allow Subsurface to check for updates - and if I accept then it asks me<br>
again, every time I start Subsurface. Only by declining can I get rid of<br>
the dialog.<br>
<br>
That's not what we want, is it? :-/<br>
<br>
This is caused by your implementation losing sight of the fact that there<br>
are THREE states. The DontCheckForUpdates can be unset, true, or false.<br>
Your code turns this into a binary case. I added a commit to fix that.<br>
<br>
I also noticed that when I make changes to the settings and then click<br>
apply, I am told that in order to change language I had to restart<br>
Subsurface (I didn't change the language...) and then when I clicked OK<br>
Subsurface crashed. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I haven't had time to investigate this one, yet.<br></blockquote><div><br></div><div>I'm looking into it - it's a bit exausting, I'm checking all preferences to see if they apply / restore, also I'm adding a unittest for the preferences.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Nevertheless, in order to get more people to test and hopefully fix<br>
things, I'll push all this to master.<br>
<span class="HOEnZb"><font color="#888888"><br>
/D<br>
</font></span></blockquote></div><br></div></div>