lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Implement straight flags in scheme


From: hanwenn
Subject: Implement straight flags in scheme
Date: Sun, 14 Dec 2008 02:44:52 +0000

Reviewers: Reinhold,


http://codereview.appspot.com/11046/diff/8/11
File scm/flag-styles.scm (left):

http://codereview.appspot.com/11046/diff/8/11#oldcode17
Line 17: ;; (define-public (add-stroke-straight stencil dir
stroke-style)
drop commented stuff.

http://codereview.appspot.com/11046/diff/8/11
File scm/flag-styles.scm (right):

http://codereview.appspot.com/11046/diff/8/11#newcode18
Line 18: (let* ((udmult (if stem-up? 1 -1))
fix naming

http://codereview.appspot.com/11046/diff/8/11#newcode28
Line 28: (let* ((conversion-constant (/ (atan 1 1) 45))
define PI = 3.141.. somewhere and use that.

split this in a polar -> rect function and a degree -> radian function.

http://codereview.appspot.com/11046/diff/8/11#newcode36
Line 36: (define (buildflag flag-stencil remain curr-stencil spacing)
doc string

http://codereview.appspot.com/11046/diff/8/11#newcode57
Line 57: (factor (if (number? fs) (expt 1.12246 fs) 1))
look for magstep to unhardcoded this

http://codereview.appspot.com/11046/diff/8/11#newcode61
Line 61: (up-length (+ (* upflag-length factor) half-stem-thickness))
can you make the up/down distinction generic? Usually up=1, down= -1, so
usually you can just do (* direction thick) to have a direction
dependent value.

http://codereview.appspot.com/11046/diff/8/11#newcode80
Line 80: (if (null? stroke-style)
use explicit check

 (eq? stroke-style 'grace)

(or whatever)

Description:
Implement straight flags in scheme

Please review this at http://codereview.appspot.com/11046

Affected files:
  M Documentation/topdocs/NEWS.tely
  A input/regression/flags-straight.ly
  M scm/flag-styles.scm






reply via email to

[Prev in Thread] Current Thread [Next in Thread]