----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100236/#review633 -----------------------------------------------------------
src/urlbar/completionwidget.cpp <http://git.reviewboard.kde.org/r/100236/#comment461> Please, use Q_FOREACH instead src/urlbar/listitem.cpp <http://git.reviewboard.kde.org/r/100236/#comment462> Is this really necessary? Comments, please. src/urlbar/listitem.cpp <http://git.reviewboard.kde.org/r/100236/#comment463> Not sure I understood what happened with the extractQuery function.. :) - Andrea On 2010-12-23 16:15:20, Lionel Chauvin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100236/ > ----------------------------------------------------------- > > (Updated 2010-12-23 16:15:20) > > > Review request for rekonq. > > > Summary > ------- > > update the completion list when the user switch search engine > > > Diffs > ----- > > src/opensearch/opensearchmanager.h df8e536 > src/opensearch/opensearchmanager.cpp 4ee1de3 > src/opensearch/searchengine.cpp 92b3293 > src/urlbar/completionwidget.h 6545b11 > src/urlbar/completionwidget.cpp 087a10a > src/urlbar/listitem.h 0d66a12 > src/urlbar/listitem.cpp d82613f > src/urlbar/urlresolver.h e6ca770 > src/urlbar/urlresolver.cpp 864eca0 > > Diff: http://git.reviewboard.kde.org/r/100236/diff > > > Testing > ------- > > > Thanks, > > Lionel > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
