AK: Stream operators for String for generic streams.

I think this should really be a member function of InputStream instead,
but I don't want to include String in Stream.h. This will do for now...

1 files changed, 22 insertions, 18 deletions

diff --git a/AK/String.h b/AK/String.h
index 52c006e0be..1fcd6de7f1 100644
--- a/AK/String.h
+++ b/AK/String.h
@@ -29,6 +29,7 @@
 #include <AK/Forward.h>
 #include <AK/RefPtr.h>
 #include <AK/Stream.h>
+#include <AK/StringBuilder.h>
 #include <AK/StringImpl.h>
 #include <AK/StringUtils.h>
 #include <AK/Traits.h>
@@ -277,26 +278,29 @@ bool operator<=(const char*, const String&);
 
 String escape_html_entities(const StringView& html);
 
-inline InputMemoryStream& operator>>(InputMemoryStream& stream, String& string)
+inline InputStream& operator>>(InputStream& stream, String& string)
 {
-    // FIXME: There was some talking about a generic lexer class?
-
-    const auto start = stream.offset();
-
-    while (!stream.eof() && stream.m_bytes[stream.m_offset]) {
-        ++stream.m_offset;
+    StringBuilder builder;
+
+    for (;;) {
+        if (stream.eof()) {
+            string = nullptr;
+
+            // FIXME: We need an InputStream::set_error_flag method.
+            stream.discard_or_error(1);
+            return stream;
+        }
+
+        char next_char;
+        stream >> next_char;
+
+        if (next_char) {
+            builder.append(next_char);
+        } else {
+            string = builder.to_string();
+            return stream;
+        }
     }
-
-    if (stream.eof()) {
-        stream.m_error = true;
-        stream.m_offset = start;
-        string = nullptr;
-    } else {
-        string = String { stream.bytes().slice(start, stream.offset() - start) };
-        ++stream.m_offset;
-    }
-
-    return stream;
 }
 
 }