From b148bc31697698959ab02c95e45f87e3b31fa565 Mon Sep 17 00:00:00 2001
From: Joel Klinghed <the_jk@spawned.biz>
Date: Wed, 17 Sep 2025 23:09:18 +0200
Subject: fixup! uline: Add unicode line reader

---
 src/uline.cc | 23 ++++++++---------------
 src/uline.hh | 16 ++++------------
 2 files changed, 12 insertions(+), 27 deletions(-)

(limited to 'src')

diff --git a/src/uline.cc b/src/uline.cc
index 21927b3..7d4c7c7 100644
--- a/src/uline.cc
+++ b/src/uline.cc
@@ -3,6 +3,7 @@
 #include "check.hh"
 
 #include <algorithm>
+#include <array>
 #include <cassert>
 #include <cstdint>
 #include <cstring>
@@ -108,7 +109,7 @@ class UnicodeReader {
   std::expected<size_t, io::ReadError> fill() {
     auto ret = reader_->read(wptr_, (end_ - wptr_) * sizeof(T));
     if (ret.has_value())
-      wptr_ += ret.value() / sizeof(T);
+      wptr_ += ret.value() / sizeof(T);  // NOLINT(bugprone-sizeof-expression)
     return ret;
   }
 
@@ -125,9 +126,7 @@ class UnicodeReader {
 
 }  // namespace
 
-namespace u8 {
-
-namespace line {
+namespace u8::line {
 
 namespace {
 
@@ -154,16 +153,12 @@ class ReaderImpl : public UnicodeReader<char, u8::Reader>,
 [[nodiscard]]
 std::unique_ptr<Reader> open(std::unique_ptr<u8::Reader> reader,
                              size_t max_len) {
-  return std::make_unique<ReaderImpl>(std::move(reader), std::move(max_len));
+  return std::make_unique<ReaderImpl>(std::move(reader), max_len);
 }
 
-}  // namespace line
-
-}  // namespace u8
+}  // namespace u8::line
 
-namespace u16 {
-
-namespace line {
+namespace u16::line {
 
 namespace {
 
@@ -190,9 +185,7 @@ class ReaderImpl : public UnicodeReader<char16_t, u16::Reader>,
 [[nodiscard]]
 std::unique_ptr<Reader> open(std::unique_ptr<u16::Reader> reader,
                              size_t max_len) {
-  return std::make_unique<ReaderImpl>(std::move(reader), std::move(max_len));
+  return std::make_unique<ReaderImpl>(std::move(reader), max_len);
 }
 
-}  // namespace line
-
-}  // namespace u16
+}  // namespace u16::line
diff --git a/src/uline.hh b/src/uline.hh
index a3bcef3..28e2936 100644
--- a/src/uline.hh
+++ b/src/uline.hh
@@ -9,9 +9,7 @@
 #include <memory>
 #include <string_view>
 
-namespace u8 {
-
-namespace line {
+namespace u8::line {
 
 class Reader {
  public:
@@ -35,13 +33,9 @@ class Reader {
 [[nodiscard]] std::unique_ptr<Reader> open(std::unique_ptr<u8::Reader> reader,
                                            size_t max_len = 8192);
 
-}  // namespace line
-
-}  // namespace u8
+}  // namespace u8::line
 
-namespace u16 {
-
-namespace line {
+namespace u16::line {
 
 class Reader {
  public:
@@ -65,8 +59,6 @@ class Reader {
 [[nodiscard]] std::unique_ptr<Reader> open(std::unique_ptr<u16::Reader> reader,
                                            size_t max_len = 8192);
 
-}  // namespace line
-
-}  // namespace u16
+}  // namespace u16::line
 
 #endif  // ULINE_HH
-- 
cgit v1.3