----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113331/#review42178 -----------------------------------------------------------
Ship it! Looks good to me too. logviewer/dates-model.cpp <http://git.reviewboard.kde.org/r/113331/#comment30723> don't do < in an enum. It's weird and liable to break. - David Edmundson On Oct. 18, 2013, 3:49 p.m., Dan Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113331/ > ----------------------------------------------------------- > > (Updated Oct. 18, 2013, 3:49 p.m.) > > > Review request for Telepathy and Leon Handreke. > > > Repository: ktp-text-ui > > > Description > ------- > > Much nicer and simpler :) > > > Diffs > ----- > > logviewer/dates-model.h 773f3b0 > logviewer/dates-model.cpp 93f4814 > > Diff: http://git.reviewboard.kde.org/r/113331/diff/ > > > Testing > ------- > > Clicked a few times and nothing crashed. > > > Thanks, > > Dan Vrátil > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
