[lmi-commits] [lmi] master 5a58eab 3/4: Resolve a marked defect: no reason to elide comma [398]

[Greg Chicares]

branch: master
commit 5a58eab1de1e40b511a5936e5b1c41da2e6c1eb2
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Resolve a marked defect: no reason to elide comma [398]
    
    It is perfectly normal to require a comma in interval notation,
    and nothing would be gained by allowing its elision.
      [begin, end)  // conventional and clear
      [begin end)   // unconventional and therefore less clear
---
 input_sequence_parser.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/input_sequence_parser.cpp b/input_sequence_parser.cpp
index e09ee38..19c0162 100644
--- a/input_sequence_parser.cpp
+++ b/input_sequence_parser.cpp
@@ -251,7 +251,7 @@ void SequenceParser::intervalic_duration()
     duration_mode tentative_begin_duration_mode = 
current_duration_scalar_mode_;
     int tentative_end_duration                  = -1;
     duration_mode tentative_end_duration_mode   = e_invalid_mode;
-    match(e_minor_separator); // TODO ?? Require this?
+    match(e_minor_separator);
     duration_scalar();
     if
         (  e_end_incl == current_token_type_