----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101174/#review2906 -----------------------------------------------------------
Looks good, just some comments. kio/kio/slavebase.cpp <http://git.reviewboard.kde.org/r/101174/#comment2500> add "We need to let the kioslave try logging in with it first", to explain the reason for this. kio/kio/slavebase.cpp <http://git.reviewboard.kde.org/r/101174/#comment2502> Why not just info = dlgInfo? This would also make this code more robust in case new fields are added to AuthInfo. kio/kio/slavebase.cpp <http://git.reviewboard.kde.org/r/101174/#comment2501> Nitpick: no space after the method name. - David On April 24, 2011, 12:10 a.m., Dawit Alemayehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101174/ > ----------------------------------------------------------- > > (Updated April 24, 2011, 12:10 a.m.) > > > Review request for kdelibs and David Faure. > > > Summary > ------- > > The is the first part of a two part patch set that changes the current > behavior of openPasswordDialog such that it will not automatically store the > password even when the user checks the "Remember password" checkbox. The > reason behind this change is to make sure incorrect or invalid password > information is not stored in the wallet. And that can only be achieved if the > ioslaves first have to verify the password information is valid by using it > to gain access to the password protected server. > > > Diffs > ----- > > kio/kio/slavebase.h f8ee99a > kio/kio/slavebase.cpp 6432edb > > Diff: http://git.reviewboard.kde.org/r/101174/diff > > > Testing > ------- > > > Thanks, > > Dawit > >
