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

Issue 306710043: Avoid a few trivial lambda function wrappers (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 6 months ago by dak
Modified:
5 years, 8 months ago
Reviewers:
simon.albrecht
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Avoid a few trivial lambda function wrappers

Patch Set 1 #

Total comments: 8

Patch Set 2 : Simon's suggestions #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -22 lines) Patch
M Documentation/snippets/transposing-pitches-with-minimum-accidentals-smart-transpose.ly View 1 chunk +1 line, -1 line 0 comments Download
M ly/articulate.ly View 4 chunks +6 lines, -6 lines 0 comments Download
M ly/music-functions-init.ly View 1 1 chunk +1 line, -3 lines 0 comments Download
M scm/chord-name.scm View 1 chunk +1 line, -1 line 0 comments Download
M scm/define-markup-commands.scm View 1 chunk +1 line, -1 line 0 comments Download
M scm/define-music-display-methods.scm View 1 chunk +1 line, -1 line 0 comments Download
M scm/documentation-lib.scm View 1 1 chunk +1 line, -2 lines 0 comments Download
M scm/modal-transforms.scm View 1 1 chunk +1 line, -3 lines 0 comments Download
M scm/music-functions.scm View 1 1 chunk +1 line, -3 lines 0 comments Download
M scm/stencil.scm View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
simon.albrecht
Good call, thanks! https://codereview.appspot.com/306710043/diff/1/ly/music-functions-init.ly File ly/music-functions-init.ly (right): https://codereview.appspot.com/306710043/diff/1/ly/music-functions-init.ly#newcode1656 ly/music-functions-init.ly:1656: coords offsets))) Now the line-breaks can ...
7 years, 6 months ago (2016-10-19 21:37:12 UTC) #1
dak
I hate having to clean up after sed scripts but it was less work just ...
7 years, 6 months ago (2016-10-20 08:42:11 UTC) #2
dak
7 years, 6 months ago (2016-10-20 08:42:39 UTC) #3
Simon's suggestions
Sign in to reply to this message.

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