Re: Review Request 127387: Refactor the backend loading code

2017-02-04 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/ --- (Updated Feb. 4, 2017, 11:18 a.m.) Status -- This change has been ma

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/ --- (Updated March 16, 2016, 1:54 p.m.) Review request for Plasma, Solid, Dan

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/ --- (Updated March 16, 2016, 1:50 p.m.) Review request for Plasma, Solid, Dan

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Sebastian Kügler
> On March 15, 2016, 11:42 p.m., David Edmundson wrote: > > src/backendmanager.cpp, line 216 > > > > > > is the name argument still used? > > > > It looks like that used to be the name of the backend to

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/#review93591 --- logic looks good and it's mostly nitpicking on my side. aut

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Sebastian Kügler
> On March 16, 2016, 11:45 a.m., Martin Gräßlin wrote: > > src/backendmanager.cpp, line 165 > > > > > > startsWith always goes with QLatin1String QString() is needed for .arg() (which QLatin1String doesn't have)

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/#review93598 --- autotests/testbackendloader.cpp (line 46)

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/#review93597 --- src/backendmanager.cpp (line 86)

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Daniel Vrátil
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/#review93638 --- +1, looks good to me. - Daniel Vrátil On March 16, 2016, 3

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/#review93639 --- Ship it! Ship It! - Martin Gräßlin On March 16, 2016, 3

Re: Review Request 127387: Refactor the backend loading code

2016-03-19 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/ --- (Updated March 16, 2016, 2:04 p.m.) Review request for Plasma, Solid, Dan

Re: Review Request 127387: Refactor the backend loading code

2016-03-15 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/#review93578 --- looks generally good. autotests/testbackendloader.cpp (line