[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5607 typedef System::rank_type

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

• Patch: review --> countdown
• Comment:

Patch on countdown for Nov 25th

---

[issues:#5607] typedef System::rank_type

Status: Started
Created: Tue Nov 19, 2019 05:35 PM UTC by Dan Eble
Last Updated: Wed Nov 20, 2019 07:13 AM UTC
Owner: Dan Eble

1. break-substitution: typedef System_range to clarify code

Call an interval of System rank a "System_range" rather than a "Slice."

Though both of these are currently Interval_t<int>, there is code that
stores System ranks in shorts. At some point, it might make sense to
change System_range to Interval_t<short> for consistency.</short></int>

1. typedef System::rank_type

Code in break-substitution.cc previously assumed that there would not
be more than 32k systems. This centralizes that assumption in System
and checks for programming errors.

https://codereview.appspot.com/581270043

---

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.

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto