<p dir="ltr">Actually all of those should be the correct type instead of QString, this was made because dirk had trouble with qml, but it's incorrect imo. </p>
<p dir="ltr">(tomaz, who has just started to do something he thinks is a nice improvement to subsurface) <br>
</p>
<div class="gmail_extra"><br><div class="gmail_quote">Em 18 de jul de 2016 19:43, "Lubomir I. Ivanov" <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>> escreveu:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 19 July 2016 at 01:08, Tim Wootton <<a href="mailto:tim@tee-jay.org.uk">tim@tee-jay.org.uk</a>> wrote:<br>
> Signed-off-by: Tim Wootton <<a href="mailto:tim@tee-jay.org.uk">tim@tee-jay.org.uk</a>><br>
> ---<br>
<br>
hello,<br>
<br>
it appears that the commit message has become a part of the commit title.<br>
<br>
other than that ACK.<br>
given "int" properties (apparently) work in Grantlee the patch is good.<br>
<br>
this small error has slipped in 6f95242f.<br>
QString(m_dive->maxcns), would threat the m_dive->maxcns as QChar and<br>
return a string consisting of a single unicode character.<br>
QString::number(m_dive->maxcns) is the alternative solution.<br>
<br>
lubomir<br>
--<br>
_______________________________________________<br>
subsurface mailing list<br>
<a href="mailto:subsurface@subsurface-divelog.org">subsurface@subsurface-divelog.org</a><br>
<a href="http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface" rel="noreferrer" target="_blank">http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface</a><br>
</blockquote></div></div>