Cleanup CursorThemeConfig remove uneeded ; and remove unused variable / method
ClosedPublic

Authored by bport on Oct 9 2019, 2:36 PM.

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bport created this revision.Oct 9 2019, 2:36 PM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 9 2019, 2:36 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
bport requested review of this revision.Oct 9 2019, 2:36 PM
davidedmundson accepted this revision.Oct 9 2019, 2:44 PM
This revision is now accepted and ready to land.Oct 9 2019, 2:44 PM
ervin requested changes to this revision.Oct 9 2019, 8:03 PM

Some might consider it a nitpick, but I like a history as clean as possible. ;-)

kcms/cursortheme/kcmcursortheme.cpp
51

Shouldn't be in that patch.

This revision now requires changes to proceed.Oct 9 2019, 8:03 PM
bport updated this revision to Diff 67604.Oct 10 2019, 1:46 PM

Remove commented include

bport marked an inline comment as done.Oct 10 2019, 1:52 PM
ervin accepted this revision.Oct 15 2019, 8:45 AM
This revision is now accepted and ready to land.Oct 15 2019, 8:45 AM
This revision was automatically updated to reflect the committed changes.