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

Issue 110060045: Issue 3958: Give examples of \applyContext usage.

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 10 months ago by Mark Polesky
Modified:
9 years, 9 months ago
Reviewers:
Keith, janek, dak
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

The documentation on \applyContext is pretty slim. I'm just mentioning some new tricks I recently learned, in the hopes that it will benefit future users/developers. http://code.google.com/p/lilypond/issues/detail?id=3958

Patch Set 1 #

Total comments: 3

Patch Set 2 : Simplify. #

Patch Set 3 : Texinfo formatting. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+121 lines, -9 lines) Patch
M Documentation/extending/programming-interface.itely View 1 2 1 chunk +121 lines, -9 lines 0 comments Download

Messages

Total messages: 5
janek
LGTM - this is helpful, thanks!
9 years, 10 months ago (2014-06-23 09:15:55 UTC) #1
Keith
Looks fine as-is. You might look at it again, and see if you like the ...
9 years, 10 months ago (2014-06-24 06:03:49 UTC) #2
dak
https://codereview.appspot.com/110060045/diff/1/Documentation/extending/programming-interface.itely File Documentation/extending/programming-interface.itely (right): https://codereview.appspot.com/110060045/diff/1/Documentation/extending/programming-interface.itely#newcode1339 Documentation/extending/programming-interface.itely:1339: @end lilypond This passage is too long. One thing ...
9 years, 10 months ago (2014-06-24 19:35:18 UTC) #3
Mark Polesky
I've rewritten the \applyContext description, and this new patch is about 40% shorter than the ...
9 years, 9 months ago (2014-07-04 10:54:35 UTC) #4
janek
9 years, 9 months ago (2014-07-04 21:07:24 UTC) #5
LGTM, thanks!
Sign in to reply to this message.

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