Re: Allow event functions from partial functions or partial markups (issue 336670043 by d...@gnu.org)

2018-03-20 Thread Carl . D . Sorensen

On 2018/03/20 20:31:48, thomasmorley651 wrote:


problem with \etc.
So I vote for putting all in 2.20.


I concur.

https://codereview.appspot.com/336670043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Allow event functions from partial functions or partial markups (issue 336670043 by d...@gnu.org)

2018-03-20 Thread thomasmorley65

On 2018/03/20 12:56:37, dak wrote:

On 2018/03/16 19:19:15, thomasmorley651 wrote:
> Can't review this one.
> Though, from description, I'd like to suggest to put up an entry in

`changes'.


Ugh.  Documentation for \etc is sort-of sparse, and the main problem

for

"Changes" is "which Changes"?  \etc in general is a 2.20 feature and

this fits

pretty well within that framework.  But it's as to yet not tested all

that much.

Apart from wrong estimations (including my own), I don't remember any
reported problem with \etc.
So I vote for putting all in 2.20.


https://codereview.appspot.com/336670043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Allow event functions from partial functions or partial markups (issue 336670043 by d...@gnu.org)

2018-03-20 Thread dak

On 2018/03/16 19:19:15, thomasmorley651 wrote:

Can't review this one.
Though, from description, I'd like to suggest to put up an entry in

`changes'.

Ugh.  Documentation for \etc is sort-of sparse, and the main problem for
"Changes" is "which Changes"?  \etc in general is a 2.20 feature and
this fits pretty well within that framework.  But it's as to yet not
tested all that much.

https://codereview.appspot.com/336670043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Allow event functions from partial functions or partial markups (issue 336670043 by d...@gnu.org)

2018-03-16 Thread dak

Reviewers: thomasmorley651,

Message:
On 2018/03/16 19:19:15, thomasmorley651 wrote:

Can't review this one.
Though, from description, I'd like to suggest to put up an entry in

`changes'.

Well, yeah.  Doesn't make a lot of sense without documentation and
regtest either.  I'll likely get them done separately.

Description:
Allow event functions from partial functions or partial markups

This gives event functions such as ^\tweak font-size 2^\markup \huge
\etc


Also contains commits:

Allow event functions from partial markup commands

This allows event functions such as ^\markup \huge #red \etc


Add Syntax::create_script

This generates post-events from generic Scheme expressions.

Please review this at https://codereview.appspot.com/336670043/

Affected files (+84, -30 lines):
  M lily/include/lily-imports.hh
  M lily/lily-imports.cc
  M lily/parser.yy
  M scm/ly-syntax-constructors.scm



___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Allow event functions from partial functions or partial markups (issue 336670043 by d...@gnu.org)

2018-03-16 Thread thomasmorley65

Can't review this one.
Though, from description, I'd like to suggest to put up an entry in
`changes'.

https://codereview.appspot.com/336670043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel