----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110313/#review32045 -----------------------------------------------------------
kurifilter-plugins/ikws/ikwsopts.cpp <http://git.reviewboard.kde.org/r/110313/#comment23879> what exactly is <nl/> ? Is this i18n specific keyword ? kurifilter-plugins/ikws/ikwsopts.cpp <http://git.reviewboard.kde.org/r/110313/#comment23880> What is the point of using a QPointer when the parent is set to NULL ? - Dawit Alemayehu On May 4, 2013, 10:50 p.m., Maarten De Meyer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110313/ > ----------------------------------------------------------- > > (Updated May 4, 2013, 10:50 p.m.) > > > Review request for KDE Runtime. > > > Description > ------- > > This fixes most of the Krazy fixes for KUrifilter-plugins. > http://www.englishbreakfastnetwork.org/krazy/reports/kde-4.x/kde-runtime/kurifilter-plugins/index.html > > > Diffs > ----- > > kurifilter-plugins/shorturi/kshorturifilter.cpp 802b325 > kurifilter-plugins/localdomain/localdomainurifilter.h 894b624 > kurifilter-plugins/ikws/searchproviderdlg.cpp eb4b31d > kurifilter-plugins/ikws/searchprovider.h 7f26ba7 > kurifilter-plugins/ikws/ikwsopts_ui.ui 42e9675 > kurifilter-plugins/ikws/ikwsopts_p.h d048f08 > kurifilter-plugins/ikws/ikwsopts.cpp 8ca80b6 > kurifilter-plugins/fixhost/fixhosturifilter.h 8603e36 > kurifilter-plugins/tests/kurifiltertest.cpp dd18eba > > Diff: http://git.reviewboard.kde.org/r/110313/diff/ > > > Testing > ------- > > Compiles. > Only 4 issues remaining. > > > Thanks, > > Maarten De Meyer > >
