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

Issue 7497046: Documents make-partial-ellipse-stencil (Closed)

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

Description

Documents make-partial-ellipse-stencil

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+50 lines, -2 lines) Patch
M scm/stencil.scm View 4 chunks +50 lines, -2 lines 1 comment Download

Messages

Total messages: 5
marc
Just one nitpick. https://codereview.appspot.com/7497046/diff/1/scm/stencil.scm File scm/stencil.scm (right): https://codereview.appspot.com/7497046/diff/1/scm/stencil.scm#newcode251 scm/stencil.scm:251: "Makes a list of angle/radius paris ...
11 years, 1 month ago (2013-03-17 16:22:29 UTC) #1
mike7
On 17 mars 2013, at 17:22, marc@hohlart.de wrote: > Just one nitpick. > > > ...
11 years, 1 month ago (2013-03-17 16:27:13 UTC) #2
janek
LGTM
11 years, 1 month ago (2013-03-17 16:28:32 UTC) #3
marc
Am 17.03.2013 17:27, schrieb mike@mikesolomon.org: > > On 17 mars 2013, at 17:22, marc@hohlart.de wrote: ...
11 years, 1 month ago (2013-03-17 16:42:35 UTC) #4
Graham Percival
11 years, 1 month ago (2013-03-21 03:14:13 UTC) #5
LGTM as long as that "Paris" typo is fixed.
Sign in to reply to this message.

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