On 14/03/16 18:45, Christian David via KDE Bugzilla wrote:
https://bugs.kde.org/show_bug.cgi?id=360500 Christian David <christian-da...@web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |christian-da...@web.de --- Comment #3 from Christian David <christian-da...@web.de> --- In void TransactionEditor::slotNumberChanged(const QString& txt): Why do you use loadText() and not setText() in you patch?
There are two separate routines involved, and one was using loadText() and the other setText(). I couldn't see any reason for the difference, and just decided to use loadText().
However, the loadText() seems unnecessary to me.
In the routine you refer to, then, yes, it does seem unnecessary, but not so in void TransactionEditor::assignNextNumber(). The first one there is necessary.
Allan
