----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110643/#review33132 -----------------------------------------------------------
kdeui/dialogs/kpassworddialog.h <http://git.reviewboard.kde.org/r/110643/#comment24501> Trailing spaces kdeui/dialogs/kpassworddialog.cpp <http://git.reviewboard.kde.org/r/110643/#comment24502> Sometimes there are spaces around parentheses, sometimes there is only one space and sometimes there's none. Style should be uniform. What's the preferred? (Mine is no spaces) - David Gil Oliva On May 25, 2013, 12:57 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110643/ > ----------------------------------------------------------- > > (Updated May 25, 2013, 12:57 p.m.) > > > Review request for KDE Frameworks. > > > Description > ------- > > Known "bugs" > - there's no clear icon in the text field anymore as I had to port away from > KLineEdit > > Diff shown is without the actual moving for easier reviewing. > > > Diffs > ----- > > kdeui/CMakeLists.txt cfa29ef > kdeui/dialogs/kpassworddialog.h e554c2a > kdeui/dialogs/kpassworddialog.cpp 168271c > kdeui/dialogs/kpassworddialog.ui 95845b9 > kdeui/tests/CMakeLists.txt cd055d5 > kdeui/tests/kpassworddialogtest.cpp b1c55ee > tier1/kwidgetsaddons/src/CMakeLists.txt df22699 > tier1/kwidgetsaddons/tests/CMakeLists.txt de5e6e4 > > Diff: http://git.reviewboard.kde.org/r/110643/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ Kde-frameworks-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
