----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124487/#review83082 -----------------------------------------------------------
Oops I've pushed this by mistake with other commit, if you think there's anything that should be changed, please reopen this and I'll revert it. - Martin Klapetek On July 28, 2015, 12:09 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124487/ > ----------------------------------------------------------- > > (Updated July 28, 2015, 12:09 p.m.) > > > Review request for KDE Frameworks and Valentin Rusu. > > > Repository: kwallet > > > Description > ------- > > [theese will be separate commits] > > In the kwalletbackend.cc, all error codes are negative, I assume > the "unknown cipher or hash" is also an error code and as such > should be negative? > > When trying to open the backend in KWalletD::pamOpen, return the > return code from the backend rather than -1. I'm not sure if there > is perhaps an intent to always return -1 here though, but returning > the actual return code seems more useful. > > > Diffs > ----- > > src/runtime/kwalletd/backend/kwalletbackend.cc 34e0419 > src/runtime/kwalletd/kwalletd.cpp c40289a > src/runtime/kwalletd/main.cpp 46ba2e6 > > Diff: https://git.reviewboard.kde.org/r/124487/diff/ > > > Testing > ------- > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
