https://bugs.kde.org/show_bug.cgi?id=360500
--- Comment #4 from allan <agande...@gmail.com> --- 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 -- You are receiving this mail because: You are the assignee for the bug.
