lilypond-devel
[Top][All Lists]
Advanced

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

Rationalize string number handling for notes and chords (issue906045)


From: n . puttock
Subject: Rationalize string number handling for notes and chords (issue906045)
Date: Mon, 12 Apr 2010 20:12:31 +0000

Hi Carl,

LGTM.

Regarding the style nitpicks, fixcc.py will sort all these out
automatically (though it does still have the annoying habit of
reformatting the trailing closing parenthesis in the
ADD_TRANSLATOR/ADD_INTERFACE macros).

Cheers,
Neil


http://codereview.appspot.com/906045/diff/1/3
File lily/articulations.cc (right):

http://codereview.appspot.com/906045/diff/1/3#newcode4
lily/articulations.cc:4: Copyright (C) 2010--2010  Carl Sorensen
<address@hidden>
2010  Carl Sorensen

(make grand-replace will fail to update otherwise)

http://codereview.appspot.com/906045/diff/1/3#newcode37
lily/articulations.cc:37: articulation_list (vector<Stream_event*>
note_events,
Stream_event *

http://codereview.appspot.com/906045/diff/1/3#newcode38
lily/articulations.cc:38: vector<Stream_event*> articulation_events,
Stream_event *

http://codereview.appspot.com/906045/diff/1/3#newcode39
lily/articulations.cc:39: const char* articulation_name)
char const *articulation_name

http://codereview.appspot.com/906045/diff/1/3#newcode58
lily/articulations.cc:58: !articulation_event && scm_is_pair (s); s =
scm_cdr (s))
indent

http://codereview.appspot.com/906045/diff/1/3#newcode63
lily/articulations.cc:63: articulation_event = art;
indent

http://codereview.appspot.com/906045/diff/1/3#newcode79
lily/articulations.cc:79: articulation_event->self_scm () :
indent

((articulation_event
  ? articulation_event->self_scm ()
  : SCM_EOL),

http://codereview.appspot.com/906045/diff/1/3#newcode86
lily/articulations.cc:86: return (articulations);
return scm_reverse (articulations);

http://codereview.appspot.com/906045/diff/1/5
File lily/include/articulations.hh (right):

http://codereview.appspot.com/906045/diff/1/5#newcode4
lily/include/articulations.hh:4: Copyright (C) 2010--2010 Carl Sorensen
<address@hidden>
2010 Carl Sorensen

http://codereview.appspot.com/906045/diff/1/5#newcode26
lily/include/articulations.hh:26: SCM articulation_list
(vector<Stream_event*> notes,
Stream_event *

http://codereview.appspot.com/906045/diff/1/5#newcode27
lily/include/articulations.hh:27: vector<Stream_event*> articulations,
Stream_event *

http://codereview.appspot.com/906045/diff/1/5#newcode28
lily/include/articulations.hh:28: const char* articulation_name);
char const *articulation_name

http://codereview.appspot.com/906045/diff/1/6
File lily/tab-note-heads-engraver.cc (right):

http://codereview.appspot.com/906045/diff/1/6#newcode83
lily/tab-note-heads-engraver.cc:83: SCM tab_notes =
ly_cxx_vector_to_list(note_events_);
ly_cxx_vector_to_list (note_events_);

http://codereview.appspot.com/906045/diff/1/6#newcode87
lily/tab-note-heads-engraver.cc:87: string_fret_finger = scm_call_3
(proc,
indent

http://codereview.appspot.com/906045/show




reply via email to

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