Re: Review Request 129390: KRecentFilesAction: remove unnecessary copy

2016-11-13 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129390/#review100816 --- src/krecentfilesaction.cpp (line 99)

Re: Review Request 129390: KRecentFilesAction: remove unnecessary copy

2016-11-13 Thread Elvis Angelaccio
> On Nov. 13, 2016, 12:22 p.m., Anthony Fieroni wrote: > > src/krecentfilesaction.cpp, line 191 > > > > > > removeAction(selectableActionGroup()->actions().first())->deleteLater() > > > > This should wor

Re: Review Request 129390: KRecentFilesAction: remove unnecessary copy

2016-11-13 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129390/#review100814 --- src/krecentfilesaction.cpp (line 185)

Re: Review Request 129390: KRecentFilesAction: remove unnecessary copy

2016-11-13 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129390/ --- (Updated Nov. 13, 2016, 11:45 a.m.) Status -- This change has been d