> On March 31, 2012, 8:08 p.m., Raphael Kubo da Costa wrote: > > Could you detail a bit more what was wrong with the way the output is/was > > being parsed?
I'll post it in few hours, sorry for not posting it when I made review request - Sergey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104439/#review12073 ----------------------------------------------------------- On March 31, 2012, 8:16 p.m., Sergey Borovkov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104439/ > ----------------------------------------------------------- > > (Updated March 31, 2012, 8:16 p.m.) > > > Review request for KDE Base Apps. > > > Description > ------- > > Very simple patch that fixes Bug 156396. On some systems loop was just going > forever because of the way parsing chfn output was written. > > > This addresses bug https://bugs.kde.org/show_bug.cgi?id=156396. > > http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=156396 > > > Diffs > ----- > > kdepasswd/kcm/chfnprocess.cpp 9f75d4a > > Diff: http://git.reviewboard.kde.org/r/104439/diff/ > > > Testing > ------- > > Tested it on Mandriva 2011 and Suse 12.2 with KDE 4.8.1 > > > Thanks, > > Sergey Borovkov > >
