<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug 27, 2016 at 4:46 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 12:36:25PM -0700, Dirk Hohndel wrote:<br>
><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>
> It applies nicely. I did my usual rewriting of the commit messages and a<br>
> couple of tiny whitespace fixes.<br>
><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>
> 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></span></blockquote><div><br></div><div>That was part of the fixes that I was implementing - since I took just too long to send those patches I figured it would be better to send them and start working on the fixes instead of holding them here for more time.<br> <span class=""></span><br><span class=""></span></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> 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.</span><span class=""><br></span></blockquote><div><br></div><div>didn't noticed the Three states - ups.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
</span>Oh, and I noticed that you didn't make the corresponding changes for<br>
Subsurface-mobile to keep compiling, so I added those as well.<br></blockquote><div><br></div><div>I finished installing all the android related packages yere yesterday to be able to work on all subsurfaces, so yes, I'll do them too.<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>
</font></span></blockquote></div><br></div></div>