diff options
| author | Joel Klinghed <the_jk@spawned.biz> | 2025-09-22 23:38:21 +0200 |
|---|---|---|
| committer | Joel Klinghed <the_jk@spawned.biz> | 2025-09-22 23:38:21 +0200 |
| commit | ce271f82f16ee89a18e7bfc9ed8eab7cbd6f37bc (patch) | |
| tree | 3e568faf83ae750aa244cca87b55951c7401ef03 /src/uline.hh | |
| parent | 50348284f5d82ccfd65b0c803ba0ba895912ceff (diff) | |
Change io::Reader and company to return ReadError::Eof instead of 0.
It's debatable if Eof should be considered an error or not.
But it is pretty clear it generally is a special response that
needs special handling, so easier to keep with the unexpected lot.
Also keeps better at higher abstraction levels, such as the line
reader.
Diffstat (limited to 'src/uline.hh')
| -rw-r--r-- | src/uline.hh | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/uline.hh b/src/uline.hh index 28e2936..e6ad9c1 100644 --- a/src/uline.hh +++ b/src/uline.hh @@ -1,8 +1,7 @@ #ifndef ULINE_HH #define ULINE_HH -#include "line.hh" // IWYU pragma: export -#include "uio.hh" // IWYU pragma: export +#include "uio.hh" // IWYU pragma: export #include <cstddef> #include <expected> @@ -17,7 +16,7 @@ class Reader { // Returned view is only valid until next call to read. [[nodiscard]] - virtual std::expected<std::string_view, ::line::ReadError> read() = 0; + virtual std::expected<std::string_view, io::ReadError> read() = 0; // Starts at zero. Returns next line. // So, before first read it is zero, after first read it is one. @@ -43,7 +42,7 @@ class Reader { // Returned view is only valid until next call to read. [[nodiscard]] - virtual std::expected<std::u16string_view, ::line::ReadError> read() = 0; + virtual std::expected<std::u16string_view, io::ReadError> read() = 0; // Starts at zero. Returns next line. // So, before first read it is zero, after first read it is one. |