[tor-commits] [tor/master] Merge remote-tracking branch 'onionk/rust-protocommas1'

[nickm at torproject.org]

commit a05a113062459faf7c02a7f661542b14ea5ade20
Merge: 62401812c b91bc1bab
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri Oct 19 14:20:43 2018 -0400

    Merge remote-tracking branch 'onionk/rust-protocommas1'

 changes/bug27197              |  3 +++
 src/rust/protover/protoset.rs | 33 ++++++++++++++++-----------------
 2 files changed, 19 insertions(+), 17 deletions(-)

diff --cc src/rust/protover/protoset.rs
index 3bfa7815f,4f603a33c..aa8d243ba
--- a/src/rust/protover/protoset.rs
+++ b/src/rust/protover/protoset.rs
@@@ -366,20 -347,15 +370,15 @@@ impl FromStr for ProtoSet 
          let pieces: ::std::str::Split<char> = version_string.split(',');
  
          for p in pieces {
-             if p.is_empty() {
-                 continue;
-             } else if p.contains('-') {
+             if p.contains('-') {
                  let mut pair = p.splitn(2, '-');
  
 -                let low  = pair.next().ok_or(ProtoverError::Unparseable)?;
 +                let low = pair.next().ok_or(ProtoverError::Unparseable)?;
                  let high = pair.next().ok_or(ProtoverError::Unparseable)?;
  
 -                let lo: Version =  low.parse().or(Err(ProtoverError::Unparseable))?;
 +                let lo: Version = low.parse().or(Err(ProtoverError::Unparseable))?;
                  let hi: Version = high.parse().or(Err(ProtoverError::Unparseable))?;
  
-                 if lo == u32::MAX || hi == u32::MAX {
-                     return Err(ProtoverError::ExceedsMax);
-                 }
                  pairs.push((lo, hi));
              } else {
                  let v: u32 = p.parse().or(Err(ProtoverError::Unparseable))?;