Re: Review Request 122384: Simplify and fix use of IconThemePath in StatusNotifierItems

> On Feb. 2, 2015, 3:43 p.m., Aleix Pol Gonzalez wrote: > > dataengines/statusnotifieritem/statusnotifieritemsource.cpp, line 221 > > > > > > Braces not needed. > > Marco Martin wrote: > I prefer doing like

Re: Review Request 122384: Simplify and fix use of IconThemePath in StatusNotifierItems

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122384/#review75226 --- This patch breaks other icons like Clementine and possibly oth

Re: Review Request 122384: Simplify and fix use of IconThemePath in StatusNotifierItems

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122384/ --- (Updated Feb. 2, 2015, 3:48 p.m.) Status -- This change has been mar

Re: Review Request 122384: Simplify and fix use of IconThemePath in StatusNotifierItems

> On Feb. 2, 2015, 3:43 p.m., Aleix Pol Gonzalez wrote: > > dataengines/statusnotifieritem/statusnotifieritemsource.cpp, line 221 > > > > > > Braces not needed. I prefer doing like kdelibs everywhere requiring b

Re: Review Request 122384: Simplify and fix use of IconThemePath in StatusNotifierItems

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122384/#review75221 --- dataengines/statusnotifieritem/statusnotifieritemsource.cpp <

Re: Review Request 122384: Simplify and fix use of IconThemePath in StatusNotifierItems

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122384/#review75220 --- Ship it! Ship It! - Marco Martin On Feb. 2, 2015, 3:38 p.m

Review Request 122384: Simplify and fix use of IconThemePath in StatusNotifierItems

--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122384/ --- Review request for Plasma. Repository: plasma-workspace Description ---