- **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.
2. 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 [email protected] 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
