LibPDF: Don't consume anything other than EOL in Reader::consume_eol()

This was previously a slightly confusing API. Even when there was no EOL marker at the current location, we would still consume one byte. It will now consume either EOL or nothing at all.
cubiclove · Mar 22, 2023 · fca9da4 · fca9da4
1 parent 93062e2
commit fca9da4
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/Userland/Libraries/LibPDF/Reader.cpp b/Userland/Libraries/LibPDF/Reader.cpp
@@ -46,8 +46,11 @@ bool Reader::consume_eol()
  consume(2);
  return true;
  }
- auto consumed = consume();
- return consumed == 0xd || consumed == 0xa;
+ if (matches_eol()) {
+ consume();
+ return true;
+ }
+ return false;
 }
 
 bool Reader::consume_whitespace()