On Sat, Mar 21, 2020 at 1:41 PM Thomas Baumgart <t...@net-bembel.de> wrote:
> Eventually we will do that. Guess I have to learn how to work with those > pull requests then. I am looking forward to this. Why do you use > > d->m_value = d->m_value * MyMoneyMoney::MINUS_ONE; > instead of > d->m_value = -d->m_value; > 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. > Also, make sure to invert the sign of the MyMoneySplit::m_shares member as > well. Otherwise, your book-keeping is off ;) > Indeed, I missed that one. Thank you. Fixed and tested with an investment account. Regards, Dawid Wrobel
