On 8 January 2017 at 02:10, Dirk Hohndel <[email protected]> wrote: > Actually, I found where sync() is used. In the tests. So NAK from here > > > That was silly of me. I was even looking in testpreferences.cpp to work out how the setSomething() functions were used. I'll wear the dunce's cap for not testing the tests.
If sync() is only used in the tests, would it be better if the function were in testpreferences.cpp instead? Thanks for taking the first patch, which is the one that actually fixes a bug. Rick
_______________________________________________ subsurface mailing list [email protected] http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
