----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108640/#review26304 -----------------------------------------------------------
Ship it! dataengines/microblog/koauth.cpp <http://git.reviewboard.kde.org/r/108640/#comment20021> can just as easily be: if (wallet && the isNull is not particularly necessary. - Aaron J. Seigo On Jan. 28, 2013, 12:40 p.m., David Narváez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108640/ > ----------------------------------------------------------- > > (Updated Jan. 28, 2013, 12:40 p.m.) > > > Review request for Plasma. > > > Description > ------- > > Related to bug 312137. According to the documentation, the wallet pointer can > be null if it was denied, so we need to check for null values before the > other checks. > > > Diffs > ----- > > dataengines/microblog/koauth.cpp 7566b95 > > Diff: http://git.reviewboard.kde.org/r/108640/diff/ > > > Testing > ------- > > 1. Make sure your plasma desktop does not have "Allow Always" in your kwallet > 2. Add a microblogging plasmoid > 3. Configure your plasmoid with an identi.ca account > 4. Deny any kwallet prompt > > Before this path it would crash trying to store the access tokens. > > > Thanks, > > David Narváez > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel