> On June 13, 2014, 3:25 p.m., David Edmundson wrote: > > startkde/ksyncdbusenv/ksyncdbusenv.cpp, line 38 > > <https://git.reviewboard.kde.org/r/118730/diff/1/?file=280898#file280898line38> > > > > for (auto i = map.constBegin(); i != map.constEnd(); ++i) > > env.insert(i.key(), i.value()); > > > > > > is faster here as you're not doing a hash lookup each time.
No, doesn't work. QProcessEnvironment doesn't inherit from any containter type :-( - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118730/#review59985 ----------------------------------------------------------- On June 13, 2014, 3:01 p.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118730/ > ----------------------------------------------------------- > > (Updated June 13, 2014, 3:01 p.m.) > > > Review request for Plasma and David Edmundson. > > > Repository: plasma-workspace > > > Description > ------- > > [startkde] Introduce a ksyncdbusenv helper application > > This application is invoked by startkde once all environment variables > are set and syncs them to DBus for the activation environment. > > > Diffs > ----- > > startkde/CMakeLists.txt 3360a1c59409588ecec49e63276914b69568164e > startkde/ksyncdbusenv/CMakeLists.txt PRE-CREATION > startkde/ksyncdbusenv/ksyncdbusenv.cpp PRE-CREATION > startkde/startkde.cmake 0c1f5841eeefc1012dbdc71e1d1fde11e7511dd6 > > Diff: https://git.reviewboard.kde.org/r/118730/diff/ > > > Testing > ------- > > on log in kglobalacceld5 doesn't pick up the right environment yet, but on > manual invocation of the tool + trigger restart of kglobalacceld5 it works. > So the approach seems to work, just not yet from startkde (maybe it's too > late?) > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
