Okay, feed this to the coverage tool.

ad-freiburg · Mar 20, 2024 · 0e921fd · 0e921fd
1 parent e5f6a57
commit 0e921fd
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/src/parser/TripleComponent.h b/src/parser/TripleComponent.h
@@ -224,9 +224,8 @@ class TripleComponent {
       // If `toValueId` could not convert to `Id`, we have a string, which we
       // look up in (and potentially add to) our local vocabulary.
       AD_CORRECTNESS_CHECK(isString() || isLiteral() || isIri());
-      // TODO<joka921> Code duplication + newWord can be moved.
-      // TODO<joka921> We really have to make the internalRepresentation
-      // movable.
+      // TODO<joka921> Make the internal representation of the `Literal` and
+      // `Iri` class movable.
       std::string newWord = [&]() -> std::string {
         if (isString()) {
           return getString();

diff --git a/src/parser/TurtleParser.cpp b/src/parser/TurtleParser.cpp
@@ -300,7 +300,7 @@ void TurtleParser<T>::parseIntegerConstant(std::string_view input) {
       TurtleParserIntegerOverflowBehavior::AllToDouble) {
     return parseDoubleConstant(input);
   }
-  int64_t result{0ll};
+  int64_t result{0};
   // The functions used below cannot deal with leading redundant '+' signs.
   if (input.starts_with('+')) {
     input.remove_prefix(1);

diff --git a/test/TurtleParserTest.cpp b/test/TurtleParserTest.cpp
@@ -203,6 +203,8 @@ TEST(TurtleParserTest, rdfLiteral) {
 
   literals.emplace_back(R"("42.1234"^^)"s + "<" + XSD_DOUBLE_TYPE + ">");
   expected.emplace_back(42.1234);
+  literals.emplace_back(R"("+42.2345"^^)"s + "<" + XSD_DOUBLE_TYPE + ">");
+  expected.emplace_back(42.2345);
   literals.push_back(R"("-142.321"^^)"s + "<" + XSD_DECIMAL_TYPE + ">");
   expected.emplace_back(-142.321);
   literals.push_back(R"("-142321"^^)"s + "<" + XSD_INT_TYPE + ">");