> On Sept. 1, 2015, 2:25 p.m., Martin Gräßlin wrote: > > src/client/blur.h, line 111 > > <https://git.reviewboard.kde.org/r/125015/diff/1/?file=399883#file399883line111> > > > > what is removeBlur doing? Shouldn't a delete of the Blur object be the > > same? > > Marco Martin wrote: > i just copied the same way Shadow does. > or should org_kde_kwin_blur_manager_unset be called in the dtor?
hmm might in deed be needed as we only store the region internally. - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125015/#review84698 ----------------------------------------------------------- On Sept. 1, 2015, 2 p.m., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125015/ > ----------------------------------------------------------- > > (Updated Sept. 1, 2015, 2 p.m.) > > > Review request for kwin and Plasma. > > > Repository: kwayland > > > Description > ------- > > a protocol to activate the blur behind windows and to optionally set a sub > region of the window where to apply the blur to, in case the window is shaped > > > Diffs > ----- > > autotests/client/CMakeLists.txt c2c1df2 > autotests/client/test_wayland_blur.cpp PRE-CREATION > src/client/CMakeLists.txt 7d0d38a > src/client/blur.h PRE-CREATION > src/client/blur.cpp PRE-CREATION > src/client/protocols/blur.xml PRE-CREATION > src/client/registry.h 3e9d1f4 > src/client/registry.cpp 0c1c213 > src/client/surface.h fe744bc > src/client/surface.cpp 98a3cec > src/server/CMakeLists.txt 1cf09d3 > src/server/blur_interface.h PRE-CREATION > src/server/blur_interface.cpp PRE-CREATION > src/server/display.h 4c0e0c7 > src/server/display.cpp 884d7ea > src/server/surface_interface.h 4935ff6 > src/server/surface_interface.cpp be885bd > src/server/surface_interface_p.h 062c7e7 > > Diff: https://git.reviewboard.kde.org/r/125015/diff/ > > > Testing > ------- > > > Thanks, > > Marco Martin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
