Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(179)

Issue 249270044: Python-ly to indent ../input/regression/a* (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 9 months ago by pkx166h
Modified:
7 years ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Python-ly to indent ../input/regression/a* Ran python-ly on ../input/regression/a*.ly Comes out of Issue 1432

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+147 lines, -149 lines) Patch
M input/regression/accidental.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/accidental-ancient.ly View 2 chunks +13 lines, -13 lines 0 comments Download
M input/regression/accidental-cautionary.ly View 2 chunks +8 lines, -8 lines 0 comments Download
M input/regression/accidental-collision.ly View 1 chunk +2 lines, -2 lines 0 comments Download
M input/regression/accidental-contemporary.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/accidental-double.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/accidental-grouping.ly View 1 chunk +5 lines, -5 lines 0 comments Download
M input/regression/accidental-octave.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/accidental-piano.ly View 2 chunks +7 lines, -7 lines 0 comments Download
M input/regression/accidental-suggestions.ly View 2 chunks +1 line, -3 lines 0 comments Download
M input/regression/accidental-unbroken-tie-spacing.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/add-grace-property.ly View 1 chunk +13 lines, -13 lines 0 comments Download
M input/regression/alignment-order.ly View 1 chunk +11 lines, -11 lines 0 comments Download
M input/regression/alignment-vertical-manual-setting.ly View 1 chunk +12 lines, -12 lines 1 comment Download
M input/regression/alter-broken.ly View 2 chunks +11 lines, -11 lines 2 comments Download
M input/regression/ambitus-cue.ly View 1 chunk +6 lines, -6 lines 0 comments Download
M input/regression/apply-context.ly View 2 chunks +3 lines, -3 lines 0 comments Download
M input/regression/apply-output.ly View 1 chunk +12 lines, -12 lines 0 comments Download
M input/regression/arpeggio-collision.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/arpeggio-span-one-staff.ly View 1 chunk +13 lines, -13 lines 0 comments Download
M input/regression/auto-beam-bar.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/auto-beam-exceptions.ly View 1 chunk +4 lines, -4 lines 1 comment Download
M input/regression/auto-beam-no-beam.ly View 1 chunk +3 lines, -3 lines 0 comments Download
M input/regression/auto-change.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/autobeam-show-defaults.ly View 1 chunk +4 lines, -4 lines 0 comments Download
M input/regression/autochange-inside-grace.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/autochange-with-grace.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/automatic-polyphony-context-id.ly View 1 chunk +7 lines, -7 lines 0 comments Download
M input/regression/automatic-polyphony-drumstaff.ly View 1 chunk +1 line, -1 line 0 comments Download
M input/regression/automatic-polyphony-tabstaff.ly View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
Trevor Daniels
On balance the indentation is improved, but there are a few cases which are worse, ...
8 years, 8 months ago (2015-07-04 14:21:58 UTC) #1
dak
On 2015/07/04 14:21:58, Trevor Daniels wrote: > On balance the indentation is improved, but there ...
8 years, 8 months ago (2015-07-04 22:48:08 UTC) #2
Dan Eble
8 years, 8 months ago (2015-07-04 23:45:58 UTC) #3
On 2015/07/04 22:48:08, dak wrote:
> And the source code of the regtests is not really significant documentation
> anyway.  It does not seem worthwhile to mess up the history of those files
with
> something that seems somewhat ephemeral.

+1
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b