Debug and new tests:

- Fix HalfMove memory leak
- Fix integer overflow
- Update combined tests

src/HalfMove.cpp

@@ -5,6 +5,7 @@ namespace pgnp {
 HalfMove::HalfMove() : count(-1), isBlack(false), MainLine(NULL) {}
 
 HalfMove::~HalfMove() {
+  delete MainLine;
   for (auto *move : variations) {
     delete move;
   }

src/PGN.cpp

@@ -45,7 +45,7 @@ void PGN::ParseNextGame() {
   if (IS_EOF) {
     throw NoGameFound();
   }
-  int loc = NextNonBlank(LastGameEndLoc);
+  long loc = NextNonBlank(LastGameEndLoc);
   if (IS_EOF) {
     throw NoGameFound();
   }

tests/combined.cpp

@@ -40,4 +40,35 @@ TEST_CASE("Hartwig PGN", "[combined/hartwig]") {
           "was because things were in such a rut I would only be cheered by "
           "winning in crushing style.  Thankfully it worked!");
   }
+}
+
+TEST_CASE("Kramnik PGN", "[combined/kramnik]") {
+  // PGN source: https://www.angelfire.com/games3/smartbridge/
+
+  pgnp::PGN pgn;
+  pgn.FromFile("pgn_files/combined/kramnik.pgn");
+
+  // Count games
+  REQUIRE_NOTHROW([&]() {
+    char i = 0;
+    try {
+      while (true) {
+        pgn.ParseNextGame();
+        i++;
+      }
+    } catch (const NoGameFound &e) {
+      CHECK(i == 40);
+    }
+  }());
+
+  SECTION("Check comments of a game") {
+    pgnp::PGN pgn;
+    pgn.FromFile("pgn_files/combined/kramnik.pgn");
+    pgn.ParseNextGame(); // Load game 1
+
+    HalfMove *m = new HalfMove();
+    pgn.GetMoves(m);
+    CHECK(m->comment ==
+          "E32: Nimzo-Indian: Classical (4 Qc2): 4...0-0");
+  }
 }