[effects/screenshot] Use override keyword
ClosedPublic

Authored by zzag on Oct 16 2018, 9:48 PM.

Details

Test Plan

Compiles.

Diff Detail

Repository
R108 KWin
Branch
effects-screenshot-override
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 3967
Build 3985: arc lint + arc unit
zzag created this revision.Oct 16 2018, 9:48 PM
Restricted Application added a project: KWin. · View Herald TranscriptOct 16 2018, 9:48 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Oct 16 2018, 9:48 PM
davidedmundson accepted this revision.Oct 16 2018, 11:00 PM
davidedmundson added a subscriber: davidedmundson.

I think there's a clazy script that does this automagically

This revision is now accepted and ready to land.Oct 16 2018, 11:00 PM
zzag added a comment.EditedOct 16 2018, 11:07 PM

I think there's a clazy script that does this automagically

FWIW, some time ago, I submitted a patch to use override keyword in effects, MF didn't like that.

This revision was automatically updated to reflect the committed changes.