lint: remove reinterpret_cast lint

outfoxxed · outfoxxed · commit 00ad94539859 · 2024-12-06T20:07:51.000-08:00
Unhelpful.
diff --git a/.clang-tidy b/.clang-tidy
@@ -17,6 +17,7 @@ Checks: >
   -cppcoreguidelines-avoid-goto,
   -cppcoreguidelines-pro-bounds-array-to-pointer-decay,
   -cppcoreguidelines-avoid-do-while,
+  -cppcoreguidelines-pro-type-reinterpret-cast,
   google-global-names-in-headers,
   google-readability-casting,
   google-runtime-int,
diff --git a/src/core/logging.cpp b/src/core/logging.cpp
@@ -486,22 +486,22 @@ void WriteBuffer::writeBytes(const char* data, qsizetype length) {
 }
 
 void WriteBuffer::writeU8(quint8 data) {
-	this->writeBytes(reinterpret_cast<char*>(&data), 1); // NOLINT
+	this->writeBytes(reinterpret_cast<char*>(&data), 1);
 }
 
 void WriteBuffer::writeU16(quint16 data) {
 	data = qToLittleEndian(data);
-	this->writeBytes(reinterpret_cast<char*>(&data), 2); // NOLINT
+	this->writeBytes(reinterpret_cast<char*>(&data), 2);
 }
 
 void WriteBuffer::writeU32(quint32 data) {
 	data = qToLittleEndian(data);
-	this->writeBytes(reinterpret_cast<char*>(&data), 4); // NOLINT
+	this->writeBytes(reinterpret_cast<char*>(&data), 4);
 }
 
 void WriteBuffer::writeU64(quint64 data) {
 	data = qToLittleEndian(data);
-	this->writeBytes(reinterpret_cast<char*>(&data), 8); // NOLINT
+	this->writeBytes(reinterpret_cast<char*>(&data), 8);
 }
 
 void DeviceReader::setDevice(QIODevice* device) { this->device = device; }
@@ -518,19 +518,19 @@ qsizetype DeviceReader::peekBytes(char* data, qsizetype length) {
 bool DeviceReader::skip(qsizetype length) { return this->device->skip(length) == length; }
 
 bool DeviceReader::readU8(quint8* data) {
-	return this->readBytes(reinterpret_cast<char*>(data), 1); // NOLINT
+	return this->readBytes(reinterpret_cast<char*>(data), 1);
 }
 
 bool DeviceReader::readU16(quint16* data) {
-	return this->readBytes(reinterpret_cast<char*>(data), 2); // NOLINT
+	return this->readBytes(reinterpret_cast<char*>(data), 2);
 }
 
 bool DeviceReader::readU32(quint32* data) {
-	return this->readBytes(reinterpret_cast<char*>(data), 4); // NOLINT
+	return this->readBytes(reinterpret_cast<char*>(data), 4);
 }
 
 bool DeviceReader::readU64(quint64* data) {
-	return this->readBytes(reinterpret_cast<char*>(data), 8); // NOLINT
+	return this->readBytes(reinterpret_cast<char*>(data), 8);
 }
 
 void EncodedLogWriter::setDevice(QIODevice* target) { this->buffer.setDevice(target); }
@@ -712,18 +712,18 @@ void EncodedLogWriter::writeVarInt(quint32 n) {
 
 bool EncodedLogReader::readVarInt(quint32* slot) {
 	auto bytes = std::array<quint8, 7>();
-	auto readLength = this->reader.peekBytes(reinterpret_cast<char*>(bytes.data()), 7); // NOLINT
+	auto readLength = this->reader.peekBytes(reinterpret_cast<char*>(bytes.data()), 7);
 
 	if (bytes[0] != 0xff && readLength >= 1) {
-		auto n = *reinterpret_cast<quint8*>(bytes.data()); // NOLINT
+		auto n = *reinterpret_cast<quint8*>(bytes.data());
 		if (!this->reader.skip(1)) return false;
 		*slot = qFromLittleEndian(n);
 	} else if ((bytes[1] != 0xff || bytes[2] != 0xff) && readLength >= 3) {
-		auto n = *reinterpret_cast<quint16*>(bytes.data() + 1); // NOLINT
+		auto n = *reinterpret_cast<quint16*>(bytes.data() + 1);
 		if (!this->reader.skip(3)) return false;
 		*slot = qFromLittleEndian(n);
 	} else if (readLength == 7) {
-		auto n = *reinterpret_cast<quint32*>(bytes.data() + 3); // NOLINT
+		auto n = *reinterpret_cast<quint32*>(bytes.data() + 3);
 		if (!this->reader.skip(7)) return false;
 		*slot = qFromLittleEndian(n);
 	} else return false;
diff --git a/src/core/util.hpp b/src/core/util.hpp
@@ -65,7 +65,7 @@ class DropEmitter {
 	template <class O>
 	DropEmitter(O* object, void (*signal)(O*))
 	    : object(object)
-	    , signal(*reinterpret_cast<void (*)(void*)>(signal)) {} // NOLINT
+	    , signal(*reinterpret_cast<void (*)(void*)>(signal)) {}
 
 	DropEmitter() = default;
 
diff --git a/src/dbus/dbusmenu/dbusmenu.cpp b/src/dbus/dbusmenu/dbusmenu.cpp
@@ -105,11 +105,11 @@ QQmlListProperty<QsMenuEntry> DBusMenuItem::children() {
 }
 
 qsizetype DBusMenuItem::childrenCount(QQmlListProperty<QsMenuEntry>* property) {
-	return reinterpret_cast<DBusMenuItem*>(property->object)->enabledChildren.count(); // NOLINT
+	return reinterpret_cast<DBusMenuItem*>(property->object)->enabledChildren.count();
 }
 
 QsMenuEntry* DBusMenuItem::childAt(QQmlListProperty<QsMenuEntry>* property, qsizetype index) {
-	auto* item = reinterpret_cast<DBusMenuItem*>(property->object); // NOLINT
+	auto* item = reinterpret_cast<DBusMenuItem*>(property->object);
 	return item->menu->items.value(item->enabledChildren.at(index));
 }
 
diff --git a/src/dbus/properties.cpp b/src/dbus/properties.cpp
@@ -37,7 +37,7 @@ QDBusError demarshallVariant(const QVariant& variant, const QMetaType& type, voi
 
 	if (variant.metaType() == type) {
 		if (type.id() == QMetaType::QVariant) {
-			*reinterpret_cast<QVariant*>(slot) = variant; // NOLINT
+			*reinterpret_cast<QVariant*>(slot) = variant;
 		} else {
 			type.destruct(slot);
 			type.construct(slot, variant.constData());
diff --git a/src/io/fileview.cpp b/src/io/fileview.cpp
@@ -281,7 +281,7 @@ void FileViewWriter::write(
 	if (shouldCancel.loadAcquire()) return;
 
 	if (doAtomicWrite) {
-		if (!reinterpret_cast<QSaveFile*>(file.get())->commit()) { // NOLINT
+		if (!reinterpret_cast<QSaveFile*>(file.get())->commit()) {
 			qmlWarning(view) << "Write of " << state.path << " failed: Atomic commit failed.";
 		}
 	}
diff --git a/src/services/greetd/connection.cpp b/src/services/greetd/connection.cpp
@@ -160,7 +160,7 @@ void GreetdConnection::onSocketReady() {
 	qint32 length = 0;
 
 	this->socket.read(
-	    reinterpret_cast<char*>(&length), // NOLINT
+	    reinterpret_cast<char*>(&length),
 	    sizeof(qint32)
 	);
 
@@ -249,7 +249,7 @@ void GreetdConnection::sendRequest(const QJsonObject& json) {
 	}
 
 	this->socket.write(
-	    reinterpret_cast<char*>(&length), // NOLINT
+	    reinterpret_cast<char*>(&length),
 	    sizeof(qint32)
 	);
 
diff --git a/src/services/notifications/dbusimage.cpp b/src/services/notifications/dbusimage.cpp
@@ -15,7 +15,7 @@ QImage DBusNotificationImage::createImage() const {
 	auto format = this->hasAlpha ? QImage::Format_RGBA8888 : QImage::Format_RGB888;
 
 	return QImage(
-	    reinterpret_cast<const uchar*>(this->data.data()), // NOLINT
+	    reinterpret_cast<const uchar*>(this->data.data()),
 	    this->width,
 	    this->height,
 	    format
diff --git a/src/services/pam/conversation.cpp b/src/services/pam/conversation.cpp
@@ -80,7 +80,7 @@ void PamConversation::onMessage() {
 		auto type = PamIpcEvent::Exit;
 
 		auto ok = this->pipes.readBytes(
-		    reinterpret_cast<char*>(&type), // NOLINT
+		    reinterpret_cast<char*>(&type),
 		    sizeof(PamIpcEvent)
 		);
 
@@ -90,7 +90,7 @@ void PamConversation::onMessage() {
 			auto code = PamIpcExitCode::OtherError;
 
 			ok = this->pipes.readBytes(
-			    reinterpret_cast<char*>(&code), // NOLINT
+			    reinterpret_cast<char*>(&code),
 			    sizeof(PamIpcExitCode)
 			);
 
@@ -113,7 +113,7 @@ void PamConversation::onMessage() {
 			PamIpcRequestFlags flags {};
 
 			ok = this->pipes.readBytes(
-			    reinterpret_cast<char*>(&flags), // NOLINT
+			    reinterpret_cast<char*>(&flags),
 			    sizeof(PamIpcRequestFlags)
 			);
 
diff --git a/src/services/pam/ipc.cpp b/src/services/pam/ipc.cpp
@@ -45,7 +45,7 @@ std::string PamIpcPipes::readString(bool* ok) const {
 	if (ok != nullptr) *ok = false;
 
 	uint32_t length = 0;
-	if (!this->readBytes(reinterpret_cast<char*>(&length), sizeof(length))) { // NOLINT
+	if (!this->readBytes(reinterpret_cast<char*>(&length), sizeof(length))) {
 		return "";
 	}
 
@@ -61,7 +61,7 @@ std::string PamIpcPipes::readString(bool* ok) const {
 
 bool PamIpcPipes::writeString(const std::string& string) const {
 	uint32_t length = string.length();
-	if (!this->writeBytes(reinterpret_cast<char*>(&length), sizeof(length))) { // NOLINT
+	if (!this->writeBytes(reinterpret_cast<char*>(&length), sizeof(length))) {
 		return false;
 	}
 
diff --git a/src/services/pam/subprocess.cpp b/src/services/pam/subprocess.cpp
@@ -127,14 +127,14 @@ void PamSubprocess::sendCode(PamIpcExitCode code) {
 	{
 		auto eventType = PamIpcEvent::Exit;
 		auto ok = this->pipes.writeBytes(
-		    reinterpret_cast<char*>(&eventType), // NOLINT
+		    reinterpret_cast<char*>(&eventType),
 		    sizeof(PamIpcEvent)
 		);
 
 		if (!ok) goto fail;
 
 		ok = this->pipes.writeBytes(
-		    reinterpret_cast<char*>(&code), // NOLINT
+		    reinterpret_cast<char*>(&code),
 		    sizeof(PamIpcExitCode)
 		);
 
@@ -176,14 +176,14 @@ int PamSubprocess::conversation(
 
 		auto eventType = PamIpcEvent::Request;
 		auto ok = delegate->pipes.writeBytes(
-		    reinterpret_cast<char*>(&eventType), // NOLINT
+		    reinterpret_cast<char*>(&eventType),
 		    sizeof(PamIpcEvent)
 		);
 
 		if (!ok) goto fail;
 
 		ok = delegate->pipes.writeBytes(
-		    reinterpret_cast<const char*>(&req), // NOLINT
+		    reinterpret_cast<const char*>(&req),
 		    sizeof(PamIpcRequestFlags)
 		);
 
diff --git a/src/services/pipewire/core.hpp b/src/services/pipewire/core.hpp
@@ -43,7 +43,7 @@ class PwObject {
 public:
 	explicit PwObject(T* object = nullptr): object(object) {}
 	~PwObject() {
-		pw_proxy_destroy(reinterpret_cast<pw_proxy*>(this->object)); // NOLINT
+		pw_proxy_destroy(reinterpret_cast<pw_proxy*>(this->object));
 	}
 
 	Q_DISABLE_COPY_MOVE(PwObject);
diff --git a/src/services/pipewire/node.cpp b/src/services/pipewire/node.cpp
@@ -459,19 +459,19 @@ PwVolumeProps PwVolumeProps::parseSpaPod(const spa_pod* param) {
 	const auto* channelsProp = spa_pod_find_prop(param, nullptr, SPA_PROP_channelMap);
 	const auto* muteProp = spa_pod_find_prop(param, nullptr, SPA_PROP_mute);
 
-	const auto* volumes = reinterpret_cast<const spa_pod_array*>(&volumesProp->value);   // NOLINT
-	const auto* channels = reinterpret_cast<const spa_pod_array*>(&channelsProp->value); // NOLINT
+	const auto* volumes = reinterpret_cast<const spa_pod_array*>(&volumesProp->value);
+	const auto* channels = reinterpret_cast<const spa_pod_array*>(&channelsProp->value);
 
 	spa_pod* iter = nullptr;
 	SPA_POD_ARRAY_FOREACH(volumes, iter) {
 		// Cubing behavior found in MPD source, and appears to corrospond to everyone else's measurements correctly.
-		auto linear = *reinterpret_cast<float*>(iter); // NOLINT
+		auto linear = *reinterpret_cast<float*>(iter);
 		auto visual = std::cbrt(linear);
 		props.volumes.push_back(visual);
 	}
 
 	SPA_POD_ARRAY_FOREACH(channels, iter) {
-		props.channels.push_back(*reinterpret_cast<PwAudioChannel::Enum*>(iter)); // NOLINT
+		props.channels.push_back(*reinterpret_cast<PwAudioChannel::Enum*>(iter));
 	}
 
 	spa_pod_get_bool(&muteProp->value, &props.mute);
diff --git a/src/services/pipewire/registry.hpp b/src/services/pipewire/registry.hpp
@@ -68,7 +68,7 @@ template <typename T, StringLiteral INTERFACE, quint32 VERSION>
 class PwBindable: public PwBindableObject {
 public:
 	T* proxy() {
-		return reinterpret_cast<T*>(this->object); // NOLINT
+		return reinterpret_cast<T*>(this->object);
 	}
 
 protected:
diff --git a/src/services/status_notifier/dbus_item_types.cpp b/src/services/status_notifier/dbus_item_types.cpp
@@ -23,23 +23,23 @@ QImage DBusSniIconPixmap::createImage() const {
 	// fix byte order if on a little endian machine
 	if (QSysInfo::ByteOrder == QSysInfo::LittleEndian) {
 		auto* newbuf = new quint32[this->data.size()];
-		const auto* oldbuf = reinterpret_cast<const quint32*>(this->data.constData()); // NOLINT
+		const auto* oldbuf = reinterpret_cast<const quint32*>(this->data.constData());
 
 		for (uint i = 0; i < this->data.size() / sizeof(quint32); ++i) {
 			newbuf[i] = qFromBigEndian(oldbuf[i]); // NOLINT
 		}
 
 		return QImage(
-		    reinterpret_cast<const uchar*>(newbuf), // NOLINT
+		    reinterpret_cast<const uchar*>(newbuf),
 		    this->width,
 		    this->height,
 		    QImage::Format_ARGB32,
-		    [](void* ptr) { delete[] reinterpret_cast<quint32*>(ptr); }, // NOLINT
+		    [](void* ptr) { delete[] reinterpret_cast<quint32*>(ptr); },
 		    newbuf
 		);
 	} else {
 		return QImage(
-		    reinterpret_cast<const uchar*>(this->data.constData()), // NOLINT
+		    reinterpret_cast<const uchar*>(this->data.constData()),
 		    this->width,
 		    this->height,
 		    QImage::Format_ARGB32

Original file line number	Diff line number	Diff line change
`@@ -105,11 +105,11 @@ QQmlListProperty<QsMenuEntry> DBusMenuItem::children() {`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`qsizetype DBusMenuItem::childrenCount(QQmlListProperty<QsMenuEntry>* property) {`
`108`		`- return reinterpret_cast<DBusMenuItem*>(property->object)->enabledChildren.count(); // NOLINT`
	`108`	`+ return reinterpret_cast<DBusMenuItem*>(property->object)->enabledChildren.count();`
`109`	`109`	`}`
`110`	`110`
`111`	`111`	`QsMenuEntry* DBusMenuItem::childAt(QQmlListProperty<QsMenuEntry>* property, qsizetype index) {`
`112`		`- auto* item = reinterpret_cast<DBusMenuItem*>(property->object); // NOLINT`
	`112`	`+ auto* item = reinterpret_cast<DBusMenuItem*>(property->object);`
`113`	`113`	`return item->menu->items.value(item->enabledChildren.at(index));`
`114`	`114`	`}`
`115`	`115`
Original file line number	Diff line number	Diff line change
`@@ -281,7 +281,7 @@ void FileViewWriter::write(`
`281`	`281`	`if (shouldCancel.loadAcquire()) return;`
`282`	`282`
`283`	`283`	`if (doAtomicWrite) {`
`284`		`- if (!reinterpret_cast<QSaveFile*>(file.get())->commit()) { // NOLINT`
	`284`	`+ if (!reinterpret_cast<QSaveFile*>(file.get())->commit()) {`
`285`	`285`	`qmlWarning(view) << "Write of " << state.path << " failed: Atomic commit failed.";`
`286`	`286`	`}`
`287`	`287`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ std::string PamIpcPipes::readString(bool* ok) const {`
`45`	`45`	`if (ok != nullptr) *ok = false;`
`46`	`46`
`47`	`47`	`uint32_t length = 0;`
`48`		`- if (!this->readBytes(reinterpret_cast<char*>(&length), sizeof(length))) { // NOLINT`
	`48`	`+ if (!this->readBytes(reinterpret_cast<char*>(&length), sizeof(length))) {`
`49`	`49`	`return "";`
`50`	`50`	`}`
`51`	`51`
`@@ -61,7 +61,7 @@ std::string PamIpcPipes::readString(bool* ok) const {`
`61`	`61`
`62`	`62`	`bool PamIpcPipes::writeString(const std::string& string) const {`
`63`	`63`	`uint32_t length = string.length();`
`64`		`- if (!this->writeBytes(reinterpret_cast<char*>(&length), sizeof(length))) { // NOLINT`
	`64`	`+ if (!this->writeBytes(reinterpret_cast<char*>(&length), sizeof(length))) {`
`65`	`65`	`return false;`
`66`	`66`	`}`
`67`	`67`