|
From: | Auto mailings of changes to Lily Issues |
Subject: | [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #4828 Allow override of NoteHead.ledger-positions |
Date: | Wed, 20 Apr 2016 13:27:35 +0000 |
To take this a step further, we could move the code in Staff_symbol::ledger_positions (in staff-symbol.cc) to a new Note_head::ledger_positions (in note-head.cc) and just make it the default callback for the NoteHead.ledger-positions property. The StaffSymbol grob and its properties and methods (that are used in that ledger_positions code) could be accessed from the NoteHead grob.
This NoteHead callback approach would be conceptually simpler (internally, in terms of the code), but I'm not sure how it would affect performance. We currently call Staff_Symbol::ledger_positions for each note head, so maybe there wouldn't be much difference? I suppose it would involve an additional conversion from C to Scheme and back to C since the result of the callback would be a scheme value...
[issues:#4828] Allow override of NoteHead.ledger-positions
Status: Started
Created: Wed Apr 20, 2016 01:07 PM UTC by Paul Morris
Last Updated: Wed Apr 20, 2016 01:07 PM UTC
Owner: Paul Morris
Allow override of NoteHead.ledger-positions
Can be a literal list of positions:
\once \override NoteHead.ledger-positions = #'(...)
Or a scheme procedure that returns such a list:
\override NoteHead.ledger-positions = #(lambda (grob) ...)
http://codereview.appspot.com/295970043
Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
------------------------------------------------------------------------------ Find and fix application performance issues faster with Applications Manager Applications Manager provides deep performance insights into multiple tiers of your business applications. It resolves application problems quickly and reduces your MTTR. Get your free trial! https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________ Testlilyissues-auto mailing list address@hidden https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Prev in Thread] | Current Thread | [Next in Thread] |