Implement support for trimming leading whitespace
ClosedPublic

Authored by sandsmark on Oct 20 2017, 1:53 PM.

Details

Summary

Ironically phabricator doesn't show whitespace here, so please excuse grave spelling errors.

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.
sandsmark created this revision.Oct 20 2017, 1:53 PM
Restricted Application added a project: Konsole. · View Herald TranscriptOct 20 2017, 1:53 PM

Fine with me as long as you've tested it; in the GUI, it should be above the trailing option and in the code it should be before the trailing option

sandsmark updated this revision to Diff 21112.Oct 22 2017, 10:38 AM

Re-ordered the code, have a separate patch for the arguments.

sandsmark updated this revision to Diff 21113.Oct 22 2017, 10:39 AM

Uploaded the wrong patch...

hindenburg accepted this revision.Oct 25 2017, 1:00 PM

thanks

This revision is now accepted and ready to land.Oct 25 2017, 1:00 PM

Can you commit these patches that you want in 17.12 in the next week as the cut-off is soon.

This revision was automatically updated to reflect the committed changes.