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

Issue 3326042: Add harmonic by fret and by ratio to tablatures (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 5 months ago by Carl
Modified:
13 years, 4 months ago
Reviewers:
marc, pls
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Add harmonic by fret and by ratio to tablatures Code written by Marc Hohl (All the good stuff) Code integrated by Carl Sorensen (Any mistakes are his responsibility)

Patch Set 1 : Base patch set #

Unified diffs Side-by-side diffs Delta from patch set Stats (+189 lines, -44 lines) Patch
A input/regression/tablature-harmonic-functions.ly View 1 chunk +50 lines, -0 lines 0 comments Download
M ly/engraver-init.ly View 1 chunk +0 lines, -2 lines 0 comments Download
M ly/music-functions-init.ly View 1 chunk +27 lines, -1 line 0 comments Download
M scm/tablature.scm View 2 chunks +112 lines, -41 lines 0 comments Download

Messages

Total messages: 5
Carl
I've taken Marc's harmonic.ly and moved the scheme functions into scm/tablature.scm, the music functions into ...
13 years, 5 months ago (2010-11-28 04:23:11 UTC) #1
marc
Am 28.11.2010 05:23, schrieb Carl.D.Sorensen@gmail.com: > Reviewers: marc, p.l.schmidt_gmx.de, > > Message: > I've taken ...
13 years, 5 months ago (2010-11-28 09:22:38 UTC) #2
pls
Am 28.11.2010 um 10:22 schrieb Marc Hohl: > Am 28.11.2010 05:23, schrieb Carl.D.Sorensen@gmail.com: >> Reviewers: ...
13 years, 5 months ago (2010-11-28 09:42:42 UTC) #3
Carl
I forgot to mention that this patch applies on top of http://codereview.appspot.com/2723043/ Thanks, Carl
13 years, 5 months ago (2010-11-28 14:07:07 UTC) #4
Carl
13 years, 4 months ago (2010-12-03 19:03:56 UTC) #5
This code has now been integrated in the tab-tie-follow-engraver changes, due to
my mistakes in git.

Please see http://codereview.appspot.com/2723043/

Thanks,

Carl
Sign in to reply to this message.

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