AK: Standardize the behaviour of GenericLexer::consume_until overloads

Before this commit all consume_until overloads aside from the Predicate
one would consume (and ignore) the stop char/string, while the
Predicate overload would not, in order to keep behaviour consistent,
the other overloads no longer consume the stop char/string as well.

1 files changed, 2 insertions, 0 deletions

diff --git a/Userland/Utilities/lsof.cpp b/Userland/Utilities/lsof.cpp
index 2646173dac..ab6fd4f1c4 100644
--- a/Userland/Utilities/lsof.cpp
+++ b/Userland/Utilities/lsof.cpp
@@ -31,6 +31,7 @@ static bool parse_name(StringView name, OpenFile& file)
 {
     GenericLexer lexer(name);
     auto component1 = lexer.consume_until(':');
+    lexer.ignore();
 
     if (lexer.tell_remaining() == 0) {
         file.name = component1;
@@ -50,6 +51,7 @@ static bool parse_name(StringView name, OpenFile& file)
             }
 
             auto component3 = lexer.consume_until(')');
+            lexer.ignore();
             if (lexer.tell_remaining() != 0) {
                 dbgln("parse_name: expected EOF");
                 return false;