[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #4865 Move translator initializations to X::boot ()

[Auto mailings of changes to Lily Issues]

• status: Fixed --> Verified

---

[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