----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103281/#review8571 -----------------------------------------------------------
I'm being your worst nightmare on that plugins/haze/yahoo-main-options-widget.h <http://git.reviewboard.kde.org/r/103281/#comment7266> I think this could safely not be virtual plugins/haze/yahoo-main-options-widget.cpp <http://git.reviewboard.kde.org/r/103281/#comment7268> Nitpick: do you think it would make more sense to grep for @yahoo instead of just @? Although it's safe to assume @ is a wildcard, I am wondering if haze allows people to log in with usernames such as user@otherservice. plugins/haze/yahoo-main-options-widget.cpp <http://git.reviewboard.kde.org/r/103281/#comment7267> You could use a more elegant (and readable) approach by removing the remove (haha) and using .left(). So, parameterModel()->setData(index, account.left(atIndex), Qt::EditRole); - Dario Freddi On Nov. 28, 2011, 1:01 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103281/ > ----------------------------------------------------------- > > (Updated Nov. 28, 2011, 1:01 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > Haze requires yahoo! account to be without the @yahoo.com part. This patch > removes that part if user adds it. It's imho better to do this than to put a > big scary WARNING! above the line edit. Our software appears smart. > > > This addresses bug 287190. > http://bugs.kde.org/show_bug.cgi?id=287190 > > > Diffs > ----- > > plugins/haze/yahoo-main-options-widget.h 592e582 > plugins/haze/yahoo-main-options-widget.cpp 25d163e > > Diff: http://git.reviewboard.kde.org/r/103281/diff/diff > > > Testing > ------- > > Successfully added a yahoo! account and connected. > > > Thanks, > > Martin Klapetek > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
