[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Color and/or parenthesize single dots in fret-diagrams (issue 102450
From: |
marc |
Subject: |
Re: Color and/or parenthesize single dots in fret-diagrams (issue 102450043 by address@hidden) |
Date: |
Sun, 15 Jun 2014 07:46:22 +0000 |
Hi Harm,
great work, mostly LGTM, just two small nitpicks.
https://codereview.appspot.com/102450043/diff/1/scm/fret-diagrams.scm
File scm/fret-diagrams.scm (right):
https://codereview.appspot.com/102450043/diff/1/scm/fret-diagrams.scm#newcode119
scm/fret-diagrams.scm:119: ;; parantheses
typo: parentheses or -is
https://codereview.appspot.com/102450043/diff/1/scm/fret-diagrams.scm#newcode1096
scm/fret-diagrams.scm:1096: The values for @var{string-number},
@var{fret-number}, and @var{finger}
This description is not precise IMHO. The first values have to be in
that order.
Do I have to code a value for finger before I use one ore more of the
optional arguments?
Would (place-fret 1 1 parenthesized) work?
https://codereview.appspot.com/102450043/