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

Issue 13400046: Metafont formatting instructions (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 7 months ago by PhilEHolmes
Modified:
10 years, 7 months ago
Reviewers:
dak, lemzwerg
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

An attempt to codify and illustrate how to indent LilyPond's metafont formatting rules.

Patch Set 1 #

Total comments: 1

Patch Set 2 : 8 character tabs #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -6 lines) Patch
M Documentation/contributor/feta-font.itexi View 1 1 chunk +14 lines, -6 lines 1 comment Download

Messages

Total messages: 6
PhilEHolmes
Please review
10 years, 7 months ago (2013-09-10 11:38:52 UTC) #1
lemzwerg
https://codereview.appspot.com/13400046/diff/1/Documentation/contributor/feta-font.itexi File Documentation/contributor/feta-font.itexi (right): https://codereview.appspot.com/13400046/diff/1/Documentation/contributor/feta-font.itexi#newcode133 Documentation/contributor/feta-font.itexi:133: --->--->fill z1 I think it's a bad idea to ...
10 years, 7 months ago (2013-09-10 18:08:55 UTC) #2
PhilEHolmes
8 character tabs
10 years, 7 months ago (2013-09-12 14:06:20 UTC) #3
PhilEHolmes
Please review if needed. As this is the CG, I'll assume no comments = LGTM ...
10 years, 7 months ago (2013-09-12 14:08:14 UTC) #4
lemzwerg
LGTM :-)
10 years, 7 months ago (2013-09-12 14:29:56 UTC) #5
dak
10 years, 7 months ago (2013-09-12 14:31:54 UTC) #6
https://codereview.appspot.com/13400046/diff/6001/Documentation/contributor/f...
File Documentation/contributor/feta-font.itexi (right):

https://codereview.appspot.com/13400046/diff/6001/Documentation/contributor/f...
Documentation/contributor/feta-font.itexi:127: below, with @code{------->}
indicating a tab character and any other
I'd use @samp as @code here (slightly different quoting, the purported meaning
being an extract not useful as code, probably better matching its use here).
Sign in to reply to this message.

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