On 2020/05/09 18:25:03, lemzwerg wrote:
>
https://codereview.appspot.com/560030044/diff/557800043/scm/define-context-properties.scm#newcode609
> scm/define-context-properties.scm:609: merge rests when this is set to
true.")
> While you are at it, please fix indentation (i.e., no indentation for
the
> continuation line).

Acknowledged.

> LGTM.  This issue also adds some scheme code and a conversion rule –
they are
> different commits, right?

Should they really be? The scheme code (largely inspired by Harm BTW :-)
is only needed for the snippet (which will get updated through next
makelsr anyway).

As for define-context-properties.scm, it’s used in the doc so I thought
it’d make sense to have that in the same commit that the @funindex
cleanup.

V.

https://codereview.appspot.com/560030044/

Reply via email to