|
From: | Auto mailings of changes to Lily Issues |
Subject: | [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #4865 Move translator initializations to X::boot () |
Date: | Fri, 17 Jun 2016 12:46:55 +0000 |
[issues:#4865] Move translator initializations to X::boot ()
Status: Verified
Labels: Fixed_2_19_43
Created: Sun May 29, 2016 09:26 AM UTC by David Kastrup
Last Updated: Tue May 31, 2016 04:48 PM UTC
Owner: David Kastrup
Move translator initializations to X::boot ()
This reorganization of the C++ translator initializations leads to more
predictable initialization order and makes the initializations have
access to private members of the class. This is a precondition for more
restrictive access control, helping to pin down accidental references to
members of the wrong class.
Also contains commit:
Rewrite of new translator initializations
This was done by running the following Bash script:
git grep '^ADD_TRANSLATOR (' lily|sed 's/^([^:]):ADD_TRANSLATOR ( ([a-zA-Z_]+),.$/\1 \2/' |
while read file translator
do
lines="$(git grep -h '^(ADD_ACKNOWLEDGER\|ADD_END_ACKNOWLEDGER\|IMPLEMENT_TRANSLATOR_LISTENER) ('$translator, $file)"
sed -i "$(echo "$lines"|sed 's+^..$+/&/d+')" $file
sed -i "/^ADD_TRANSLATOR ($translator,/i\
void\
$translator::boot ()\
{\
$(echo "$lines"|sed 's/IMPLEMENT_TRANSLATOR_LISTENER/ADD_LISTENER/;s/^./ &/;s/$/\/')
}\
" $file
done
http://codereview.appspot.com/294650043
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.
------------------------------------------------------------------------------ What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level. Reveals which users, apps, and protocols are consuming the most bandwidth. Provides multi-vendor support for NetFlow, J-Flow, sFlow and other flows. Make informed decisions using capacity planning reports. http://sdm.link/zohomanageengine
_______________________________________________ Testlilyissues-auto mailing list address@hidden https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Prev in Thread] | Current Thread | [Next in Thread] |