fix(semver): avoid one-past-end index, close #1178 (#1201)

Author: ken-matsui

src/Semver.hpp

@@ -104,6 +104,7 @@ struct VersionLexer {
   Result<VersionToken> consumeNumOrIdent() noexcept;
   Result<VersionToken> next() noexcept;
   Result<VersionToken> peek() noexcept;
+  constexpr char curChar() const noexcept { return isEof() ? '\0' : s[pos]; }
 };
 
 struct VersionParser {

src/VersionReq.cc

@@ -107,27 +107,27 @@ struct ComparatorLexer {
 
       OptVersion ver;
       ver.major = Try(parser.parseNum());
-      if (parser.lexer.s[parser.lexer.pos] != '.') {
+      if (parser.lexer.curChar() != '.') {
         pos = parser.lexer.pos;
         return Ok(ComparatorToken{ ComparatorToken::Ver, std::move(ver) });
       }
 
       Try(parser.parseDot());
       ver.minor = Try(parser.parseNum());
-      if (parser.lexer.s[parser.lexer.pos] != '.') {
+      if (parser.lexer.curChar() != '.') {
         pos = parser.lexer.pos;
         return Ok(ComparatorToken{ ComparatorToken::Ver, std::move(ver) });
       }
 
       Try(parser.parseDot());
       ver.patch = Try(parser.parseNum());
 
-      if (parser.lexer.s[parser.lexer.pos] == '-') {
+      if (parser.lexer.curChar() == '-') {
         parser.lexer.step();
         ver.pre = Try(parser.parsePre());
       }
 
-      if (parser.lexer.s[parser.lexer.pos] == '+') {
+      if (parser.lexer.curChar() == '+') {
         parser.lexer.step();
         Try(parser.parseBuild()); // discard build metadata
       }