Menu

#5607 typedef System::rank_type

Fixed
Dan Eble
Maintainability
2019-11-25
2019-11-19
Dan Eble
No
  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

Discussion

  • Dan Eble

    Dan Eble - 2019-11-19
    • Description has changed:

    Diff:

    
    
    • Needs: -->
    • Type: --> Maintainability
     
  • Anonymous

    Anonymous - 2019-11-20
    • Patch: new --> review
     
  • Anonymous

    Anonymous - 2019-11-20

    Passes make, make check and a full make doc.

     
  • Anonymous

    Anonymous - 2019-11-23
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2019-11-23

    Patch on countdown for Nov 25th

     
  • Anonymous

    Anonymous - 2019-11-25
    • Patch: countdown --> push
     
  • Anonymous

    Anonymous - 2019-11-25

    Patch counted down - please push

     
  • Dan Eble

    Dan Eble - 2019-11-25
    • labels: --> Fixed_2_21_0
    • status: Started --> Fixed
    • Patch: push -->
     
  • Dan Eble

    Dan Eble - 2019-11-25
    commit 000d9cfc986c2b8a4f7f0ffdfbbe25aeed1bac63
    Author: Dan Eble <nine.fierce.ballads@gmail.com>
    Date:   Mon Nov 18 10:02:30 2019 -0500
    
        Issue 5607/2: typedef System::rank_type
        ...
    
    commit 1e11c0a21f2003800165b138a5eaf9e0d88a3df7
    Author: Dan Eble <nine.fierce.ballads@gmail.com>
    Date:   Mon Nov 18 10:02:30 2019 -0500
    
        Issue 5607/1: break-substitution: typedef System_range to clarify code
        ...