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

Issue 7313082: Fixes annotate spacing (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 2 months ago by MikeSol
Modified:
11 years, 1 month ago
Reviewers:
dak
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Fixes annotate spacing

Patch Set 1 #

Total comments: 1

Patch Set 2 : Changes skyline-pair::useable? to skyline-pair-and-not-empty? #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -12 lines) Patch
M lily/skyline.cc View 1 chunk +9 lines, -0 lines 0 comments Download
M scm/lily.scm View 1 1 chunk +1 line, -0 lines 0 comments Download
M scm/paper-system.scm View 1 1 chunk +1 line, -1 line 0 comments Download
A + scm/skyline.scm View 1 2 chunks +8 lines, -11 lines 0 comments Download

Messages

Total messages: 3
dak
https://codereview.appspot.com/7313082/diff/1/scm/skyline.scm File scm/skyline.scm (right): https://codereview.appspot.com/7313082/diff/1/scm/skyline.scm#newcode23 scm/skyline.scm:23: (define-public (skyline-pair::useable? skyp) So why is this not non-empty-skyline-pair? ...
11 years, 2 months ago (2013-02-20 15:27:49 UTC) #1
dak
On 2013/02/20 15:27:49, dak wrote: > https://codereview.appspot.com/7313082/diff/1/scm/skyline.scm > File scm/skyline.scm (right): > > https://codereview.appspot.com/7313082/diff/1/scm/skyline.scm#newcode23 > ...
11 years, 2 months ago (2013-02-20 15:32:22 UTC) #2
MikeSol
11 years, 2 months ago (2013-02-23 05:40:29 UTC) #3
Changes skyline-pair::useable? to skyline-pair-and-not-empty?
Sign in to reply to this message.

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