> On June 21, 2013, 8:29 p.m., David Faure wrote: > > It's all commented out, but if that makes you happy, go for it :) > > > > (yes, make the function file-static first)
Janitorial dude question: Has this been commited and you forgot to mark it as submitted or it does still need submitting? - Albert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111164/#review34837 ----------------------------------------------------------- On June 21, 2013, 6:55 p.m., Àlex Fiestas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111164/ > ----------------------------------------------------------- > > (Updated June 21, 2013, 6:55 p.m.) > > > Review request for KDE Base Apps and David Faure. > > > Description > ------- > > Make FaviIconsModule a better neighbor of the KDED community by having its > own namespace :p > > > Diffs > ----- > > lib/konq/favicons/favicons.cpp 45d6c19 > > Diff: http://git.reviewboard.kde.org/r/111164/diff/ > > > Testing > ------- > > > Thanks, > > Àlex Fiestas > >