Merge pull request #2136 from oderwat/fix-matchers
Fixed non exhaustive case by adding else
This commit is contained in:
commit
042016acf3
1 changed files with 1 additions and 1 deletions
|
|
@ -42,7 +42,7 @@ proc validEmailAddress*(s: string): bool {.noSideEffect,
|
||||||
case toLower(x)
|
case toLower(x)
|
||||||
of "com", "org", "net", "gov", "mil", "biz", "info", "mobi", "name",
|
of "com", "org", "net", "gov", "mil", "biz", "info", "mobi", "name",
|
||||||
"aero", "jobs", "museum": return true
|
"aero", "jobs", "museum": return true
|
||||||
return false
|
else: return false
|
||||||
|
|
||||||
proc parseInt*(s: string, value: var int, validRange: Slice[int]) {.
|
proc parseInt*(s: string, value: var int, validRange: Slice[int]) {.
|
||||||
noSideEffect, rtl, extern: "nmatchParseInt".} =
|
noSideEffect, rtl, extern: "nmatchParseInt".} =
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue