----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103297/#review8727 -----------------------------------------------------------
Almost there tooltips/ktooltip.cpp <http://git.reviewboard.kde.org/r/103297/#comment7336> Handle errors properly instead of using an assert. Release code won't make this work leading to $consequence. - Dario Freddi On Dec. 2, 2011, 11:21 a.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103297/ > ----------------------------------------------------------- > > (Updated Dec. 2, 2011, 11:21 a.m.) > > > Review request for Telepathy. > > > Description > ------- > > This is the result of Google Code-in student Geoffry Song's work, based on > David's code (which was stolen from system settings). The tooltips works > perfectly, the code is great quality as well, though as it is rather big > diff, I'd like to have more eyes on it. > > > Diffs > ----- > > CMakeLists.txt 68dc90c > abstract-contact-delegate.cpp 2bd0f55 > main-widget.cpp 7f0adf1 > tooltips/contacttooltip.h PRE-CREATION > tooltips/contacttooltip.cpp PRE-CREATION > tooltips/contacttooltip.ui PRE-CREATION > tooltips/ktooltip.h PRE-CREATION > tooltips/ktooltip.cpp PRE-CREATION > tooltips/ktooltipwindow.cpp PRE-CREATION > tooltips/ktooltipwindow_p.h PRE-CREATION > tooltips/tooltipmanager.h PRE-CREATION > tooltips/tooltipmanager.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/103297/diff/diff > > > Testing > ------- > > I've been using it for two days now and all hail to Geoffry, who did really > awesome job. > > > Thanks, > > Martin Klapetek > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
