Q_PROPERTYs in preferences code

2017-12-16 Thread Berthold Stoeger
Hi all, Short question: Are the numerous Q_PROPERTY macros in core/subsurface-qt/ SettingsObjectWrapper.h needed for the mobile version? If I remove them, the desktop version compiles and works just fine. Somehow this all seems redundant: Each property has a WRITE method and a NOTIFY signal, bu

Re: Warnings (was: Subsurface libdivecomputer branch merged with upstream..)

2017-12-16 Thread Berthold Stoeger
On Freitag, 15. Dezember 2017 15:04:49 CET Lubomir I. Ivanov wrote: > On 15 December 2017 at 11:24, Berthold Stoeger > > wrote: > > On Mittwoch, 13. Dezember 2017 23:34:32 CET Dirk Hohndel wrote: > >> > On Dec 13, 2017, at 11:31 AM, Berthold Stoeger > > > > Concerning the float warning, I have a

Re: Q_PROPERTYs in preferences code

2017-12-16 Thread Dirk Hohndel
I'm reasonable sure this code came from Tomaz who is our resident Qt wizard... /D > On Dec 16, 2017, at 4:26 AM, Berthold Stoeger > wrote: > > Hi all, > > Short question: Are the numerous Q_PROPERTY macros in core/subsurface-qt/ > SettingsObjectWrapper.h needed for the mobile version? If I re

Re: Warnings (was: Subsurface libdivecomputer branch merged with upstream..)

2017-12-16 Thread Lubomir I. Ivanov
On 16 December 2017 at 14:26, Berthold Stoeger wrote: > On Freitag, 15. Dezember 2017 15:04:49 CET Lubomir I. Ivanov wrote: >> On 15 December 2017 at 11:24, Berthold Stoeger >> >> wrote: >> > On Mittwoch, 13. Dezember 2017 23:34:32 CET Dirk Hohndel wrote: >> >> > On Dec 13, 2017, at 11:31 AM, Ber

Re: Q_PROPERTYs in preferences code

2017-12-16 Thread Lubomir I. Ivanov
On 16 December 2017 at 14:26, Berthold Stoeger wrote: > Somehow this all seems redundant: Each property has a WRITE method and a > NOTIFY signal, but the signal is raised in the WRITE method anyway..? Does > this mean that the signals are raised twice on mobile? > http://doc.qt.io/qt-5/qtqml-cppi