<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 26, 2016 at 8:52 AM, Lubomir I. Ivanov <span dir="ltr"><<a href="mailto:neolit123@gmail.com" target="_blank">neolit123@gmail.com</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 26 February 2016 at 12:45, Lubomir I. Ivanov <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>> wrote:<br>
> On 26 February 2016 at 12:02, Lubomir I. Ivanov <<a href="mailto:neolit123@gmail.com">neolit123@gmail.com</a>> wrote:<br>
>> On 26 February 2016 at 09:43, Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> wrote:<br>
>>> Gehad, Lubomir,<br>
>>><br>
>>> As the resident experts, can you comment?<br>
>><br>
>> i just tested the latest master quickly and the printing of weights<br>
>> and cylinders is indeed broken.<br>
>><br>
>> Tomaz did some refactoring to moving the helper Dive class to<br>
>> /subsurface-core/subsurface-qt/DiveObjectHelper.h, which could be the<br>
>> cause as it was working before.<br>
>><br>
><br>
> <a href="http://lists.subsurface-divelog.org/pipermail/subsurface/2016-January/023912.html" rel="noreferrer" target="_blank">http://lists.subsurface-divelog.org/pipermail/subsurface/2016-January/023912.html</a><br>
> "if you modify the variables in any way let me know so that i can<br>
> update the docs."<br>
><br>
> and the variables are apparently modified...<br>
><br>
<br>
</span>actually Dirk, let me know if we want to NOT break the user interface<br>
(Grantlee variables), compared to what the docs have.<br>
<br>
two options:<br>
1) go with the changes Tomaz did (break the documented variables) and<br>
add support for printing all the weights and cylinders with something<br>
like "dive.allWeights", or "dive.allCylinders", as TMK you cannot<br>
print directly a QStringList from Grantlee (via dive.weights).<br></blockquote><div><br></div><div>it should.</div><div>I'm away a bit from the development currently because of total lack of time at work, but 'm getting vacations on monday to work on subsurface.</div><div>I'll try to make it work using QStringList.</div><div><br></div><div>Tomaz</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
2) clutter the DiveObjectHelper with some weight0(), ... weightN()<br>
methods as that's how the old document variables worked.<br>
<br>
i have lost track of what version is currently released and if the<br>
regression is already in the user hands!<br>
also, i will be on IRC today if this has to be discussed further.<br>
<br>
lubomit<br>
--<br>
</blockquote></div><br></div></div>