<div dir="ltr"><div dir="ltr">On Sat, Mar 21, 2020 at 1:41 PM Thomas Baumgart <<a href="mailto:thb@net-bembel.de">thb@net-bembel.de</a>> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Eventually we will do that. Guess I have to learn how to work with those pull requests then.</blockquote><div>I am looking forward to this.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Why do you use<br>
<br>
        d->m_value = d->m_value * MyMoneyMoney::MINUS_ONE;<br>
instead of<br>
        d->m_value = -d->m_value;<br></blockquote><div><br></div><div>I saw it used in a similar fashion in a number of places, e.g. in mymoneyforecast.cpp, so I assumed it was the "KMyMoney" way to do it.  </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Also, make sure to invert the sign of the MyMoneySplit::m_shares member as well. Otherwise, your book-keeping is off ;)<br></blockquote><div><br></div><div>Indeed, I missed that one. Thank you. Fixed and tested with an investment account. </div><div><br></div><div>Regards,</div><div>Dawid Wrobel</div><div> </div></div></div>