From ebf65572dd39fa1bf6ca5b5ac32a97b14daac145 Mon Sep 17 00:00:00 2001
From: Max Kellermann <max.kellermann@gmail.com>
Date: Sat, 28 Jan 2023 07:47:44 +0100
Subject: [PATCH] util/ByteOrder: remove unnecessary operator=
The compiler can just use the uint32_t constructor instead. This was
redundant code.
---
src/util/ByteOrder.hxx | 8 --------
1 file changed, 8 deletions(-)
diff --git a/src/util/ByteOrder.hxx b/src/util/ByteOrder.hxx
index c13c3acbb..7c4ef0db4 100644
--- a/src/util/ByteOrder.hxx
+++ b/src/util/ByteOrder.hxx
@@ -329,14 +329,6 @@ public:
(uint32_t(c) << 8) | uint32_t(d);
}
- PackedBE32 &operator=(uint32_t new_value) noexcept {
- d = uint8_t(new_value);
- c = uint8_t(new_value >> 8);
- b = uint8_t(new_value >> 16);
- a = uint8_t(new_value >> 24);
- return *this;
- }
-
/**
* Reads the raw, big-endian value.
*/