|
From: | Auto mailings of changes to Lily Issues |
Subject: | [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #4842 Bring Scheme_engraver in line with other translators. |
Date: | Tue, 24 May 2016 10:19:09 +0000 |
[issues:#4842] Bring Scheme_engraver in line with other translators.
Status: Verified
Labels: Fixed_2_19_42
Created: Tue May 03, 2016 09:25 PM UTC by David Kastrup
Last Updated: Tue May 10, 2016 08:27 AM UTC
Owner: David Kastrup
Bring Scheme_engraver in line with other translators.
A whole load of commits. First uploads will be for issue 4840 and
4841 which are not in master yet.
Commits in reverse order:
Don't special-case Scheme_engraver's methods
Don't special-case Scheme_engraver's acknowledgers
Store acknowledgers in a Scheme_hash_table
Replace Translator_method et al
All of Translator_method, Translator_group_method,
Translator_group_void_method, Callback can be expressed via SCM now.
Translator_method_binding is replaced with Method_instance.
Replace Engraver_dispatch_entry with Method_instance
This also replaces a lot of C++-centric callback machinery (like the
Grob_info_callback type) with SCM-based code.
Add Method_instance class
This is a lightweight container class combining an SCM method call
with a particular instance into a C++ callable.
Add Callback0_wrapper and Callback2_wrapper class
Those are for callbacks with 0 and 2 SCM arguments, respectively. The
former are needed mainly for translator callbacks, the second for
acknowledgers.
Issue 4841: Let Scheme_hash_table::get return SCM_UNDEFINED for unknown keys
Issue 4840: Make Translator/Performer/Engraver abstract base classes
It does not make sense for them to be instantiable.
http://codereview.appspot.com/297170043
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.
------------------------------------------------------------------------------ Mobile security can be enabling, not merely restricting. Employees who bring their own devices (BYOD) to work are irked by the imposition of MDM restrictions. Mobile Device Manager Plus allows you to control only the apps on BYO-devices by containerizing them, leaving personal data untouched! https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
_______________________________________________ Testlilyissues-auto mailing list address@hidden https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Prev in Thread] | Current Thread | [Next in Thread] |