----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102654/#review6672 -----------------------------------------------------------
Ship it! lib/chat-window-style.cpp <http://git.reviewboard.kde.org/r/102654/#comment5905> QHash<InternalIdentifier, QString> would be better. (or just a QStringList and treat the enum as an int which is technically faster) lib/chat-window-style.cpp <http://git.reviewboard.kde.org/r/102654/#comment5906> Whilst I agree with your comment, lets avoid swearing in code. lib/chat-window-style.cpp <http://git.reviewboard.kde.org/r/102654/#comment5907> - David On Sept. 18, 2011, 7:53 a.m., Dominik Schmidt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102654/ > ----------------------------------------------------------- > > (Updated Sept. 18, 2011, 7:53 a.m.) > > > Review request for Telepathy. > > > Summary > ------- > > DRY ... > > Please do nitpick about style errors, I haven't done any KDE Telepathy coding > in months so it's hard for me to remember the coding style or even stick to > it :P > > > You won't notice any difference on the surface, it's just refactoring inside. > > > Diffs > ----- > > lib/chat-window-style.h 32fe4fd > lib/chat-window-style.cpp 7186759 > > Diff: http://git.reviewboard.kde.org/r/102654/diff > > > Testing > ------- > > The config ui works just fine > > > Thanks, > > Dominik > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
