Re: Review Request 128999: Add removeMenuItem to PlasmaComponents.ContextMenu

2016-09-26 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128999/ --- (Updated Sept. 26, 2016, 3:56 p.m.) Status -- This change has been

Re: Review Request 128999: Add removeMenuItem to PlasmaComponents.ContextMenu

2016-09-26 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128999/#review99540 --- Ship it! Ship It! - Kai Uwe Broulik On Sept. 23, 2016,

Re: Review Request 128999: Add removeMenuItem to PlasmaComponents.ContextMenu

2016-09-23 Thread David Rosca
> On Sept. 22, 2016, 12:31 p.m., Kai Uwe Broulik wrote: > > src/declarativeimports/plasmacomponents/qmenu.cpp, line 231 > > > > > > if (!item) return; Should be done also in other places. - David

Re: Review Request 128999: Add removeMenuItem to PlasmaComponents.ContextMenu

2016-09-23 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128999/ --- (Updated Sept. 23, 2016, 11:20 a.m.) Review request for Plasma.

Re: Review Request 128999: Add removeMenuItem to PlasmaComponents.ContextMenu

2016-09-22 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128999/#review99412 --- src/declarativeimports/plasmacomponents/qmenu.cpp (line

Review Request 128999: Add removeMenuItem to PlasmaComponents.ContextMenu

2016-09-22 Thread David Rosca
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128999/ --- Review request for Plasma. Repository: plasma-framework Description