> On March 16, 2016, 11:45 a.m., Martin Gräßlin wrote: > > src/backendmanager.cpp, line 165 > > <https://git.reviewboard.kde.org/r/127387/diff/1/?file=453202#file453202line165> > > > > startsWith always goes with QLatin1String
QString() is needed for .arg() (which QLatin1String doesn't have). - Sebastian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127387/#review93591 ----------------------------------------------------------- On March 15, 2016, 5:57 p.m., Sebastian Kügler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127387/ > ----------------------------------------------------------- > > (Updated March 15, 2016, 5:57 p.m.) > > > Review request for Plasma, Solid, Daniel Vrátil, and Martin Gräßlin. > > > Repository: libkscreen > > > Description > ------- > > Refactor the backend loading code > > Untangle the large plugin loading logic in the BackendManager static > into separate bits. This makes the code clearer and easier to auto-test. > > - listBackends() compiles a list of backends from the plugin paths > - preferredBackend() picks the backend, in this priority: > - if KSCREEN_BACKEND is set in the environment, this is the only > used method to find the backend plugin > - if platform is X11, the XRandR backend is picked > - if platform is wayland, KWayland backend is picked > - if neither is the case, QScreen backend is picked > > It does introduce a slight behavioral change: The mechanism was based on > falling through, so it would consider another backend if the logically > picked on fails to load. This is undesired behavior, however, since the > backendloader may be able to load the plugin, but that doesn't mean that > the plugin actually work. > > Parsing of the KSCREEN_BACKEND variable is kept case-insensitive. > > > autotests for new backend loading logic > > - makes sure we find plugins installed > - pick plugins from env var > > We can't sensible test all the runtime cases yet, but this at least > covers the code paths around those few lines that do runtime detection > of x11 and wayland. > > > Diffs > ----- > > autotests/CMakeLists.txt 26c7952 > autotests/testbackendloader.cpp PRE-CREATION > src/backendmanager.cpp 382ae71 > src/backendmanager_p.h 150883b > > Diff: https://git.reviewboard.kde.org/r/127387/diff/ > > > Testing > ------- > > manual runtime tests and autotests pass > > > Thanks, > > Sebastian Kügler > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
