On Feb 12, 2012, at 3:21 PM, [email protected] wrote:

> 
> http://codereview.appspot.com/5649054/diff/1/lily/stencil-expression.cc
> File lily/stencil-expression.cc (right):
> 
> http://codereview.appspot.com/5649054/diff/1/lily/stencil-expression.cc#newcode31
> lily/stencil-expression.cc:31: nargs = scm_permanent_object (scm_cons
> (scm_cons (ly_symbol2scm ("dummy"), SCM_EOL), SCM_EOL));
> dummy?
> 
> The whole stuff is totally incomprehensible.  It is not clear what is
> stored where and for what reason, and there is no documentation
> whatsoever.  It was bad before, agreed, but it has become much worse.

Han-Wen,

Is there a specific reason for using a permanent Scheme variable in the C++ 
instead of a lookup call to the lily module?  If it's just happenstance then 
I'll move these structures to Scheme and access them from the Scheme module.

Cheers,
MS


_______________________________________________
lilypond-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to