diff options
author | Richard Walters <rwalters@digitalstirling.com> | 2020-10-09 13:34:32 -0700 |
---|---|---|
committer | Richard Walters <rwalters@digitalstirling.com> | 2020-10-09 13:34:32 -0700 |
commit | 5d744a843c0fb8af675240a20f13cd48100d32f9 (patch) | |
tree | 2b8ade01bf9bbe565ee98470c3ceab1d42c6c7b6 | |
parent | d303237c82a0d81785b8218a4d6a3f18f86f5d11 (diff) |
Remove unnecessary check in IPv6 parsing
It's not possible to have encountered a double-colon
when in the COLON_BUT_NO_GROUPS_YET state.
-rw-r--r-- | src/Uri.cpp | 8 | ||||
-rw-r--r-- | src/lib.rs | 2 |
2 files changed, 3 insertions, 7 deletions
diff --git a/src/Uri.cpp b/src/Uri.cpp index 41ea1a4..ff161f3 100644 --- a/src/Uri.cpp +++ b/src/Uri.cpp @@ -270,12 +270,8 @@ namespace { case ValidationState::COLON_BUT_NO_GROUPS_YET: { if (c == ':') { - if (doubleColonEncountered) { - return false; - } else { - doubleColonEncountered = true; - state = ValidationState::AFTER_DOUBLE_COLON; - } + doubleColonEncountered = true; + state = ValidationState::AFTER_DOUBLE_COLON; } else { return false; } @@ -316,7 +316,7 @@ fn validate_ipv6_address(address: &str) -> Result<(), Error> { }, ValidationState::ColonButNoGroupsYet => { - if c != ':' || double_colon_encountered { + if c != ':' { return Err(Error::IllegalCharacter); } double_colon_encountered = true; |