I realize this issue has already been pushed and closed. But I have a question.
What is the harm of having the downcasting in the getter function? An advantage is that we can change the downcasting in one place if it is part of the member functions for the class. If not, we have to change it throughout the code base. Maintainability would seem to be a reason to keep it as it was. I'm not asking for a revert. But I would like to understand the reason it was worth it for Dan to make this change. Mostly for my own understanding of good programming practices. Thanks, Carl https://codereview.appspot.com/344010043/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
