Simplify TerminalDisplay code
ClosedPublic

Authored by tcanabrava on Mar 28 2019, 11:35 AM.

Details

Diff Detail

Repository
R319 Konsole
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
tcanabrava created this revision.Mar 28 2019, 11:35 AM
Restricted Application added a project: Konsole. · View Herald TranscriptMar 28 2019, 11:35 AM
Restricted Application added a subscriber: konsole-devel. · View Herald Transcript
tcanabrava requested review of this revision.Mar 28 2019, 11:35 AM
  • Use one liners where appropriate.
  • Remove sender() and QVariant() conversions
  • Declare + initialize, instead of setting the value later
  • Don't create unecessary temporaries
  • Extract code from huge function into a smaller one
tcanabrava retitled this revision from Use one liners where appropriate. to Simplify TerminalDisplay code .Mar 28 2019, 11:42 AM
tcanabrava added a reviewer: Konsole.
hindenburg accepted this revision.Mar 28 2019, 1:50 PM
hindenburg added a subscriber: hindenburg.

thanks

This revision is now accepted and ready to land.Mar 28 2019, 1:50 PM
This revision was automatically updated to reflect the committed changes.