From d9e7c2fb42fb8e62323f609a1c4ac8f40b8a4a88 Mon Sep 17 00:00:00 2001
From: elsid <elsid.mail@gmail.com>
Date: Tue, 4 May 2021 13:21:47 +0200
Subject: [PATCH] Replace comment by static assert

---
 components/nif/nifstream.hpp | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/components/nif/nifstream.hpp b/components/nif/nifstream.hpp
index 4d77b7326..ad0ba5da8 100644
--- a/components/nif/nifstream.hpp
+++ b/components/nif/nifstream.hpp
@@ -8,6 +8,7 @@
 #include <stdexcept>
 #include <vector>
 #include <typeinfo>
+#include <type_traits>
 
 #include <components/files/constrainedfilestream.hpp>
 #include <components/misc/endianness.hpp>
@@ -23,11 +24,9 @@ namespace Nif
 
 class NIFFile;
 
-/* 
-    readLittleEndianBufferOfType: This template should only be used with arithmetic types
-*/
 template <std::size_t numInstances, typename T> inline void readLittleEndianBufferOfType(Files::IStreamPtr &pIStream, T* dest)
 {
+    static_assert(std::is_arithmetic_v<T>, "Buffer element type is not arithmetic");
     pIStream->read((char*)dest, numInstances * sizeof(T));
     if (pIStream->bad())
         throw std::runtime_error("Failed to read little endian typed (" + std::string(typeid(T).name()) + ") buffer of "
@@ -37,11 +36,9 @@ template <std::size_t numInstances, typename T> inline void readLittleEndianBuff
             Misc::swapEndiannessInplace(dest[i]);
 }
 
-/*
-    readLittleEndianDynamicBufferOfType: This template should only be used with arithmetic types
-*/
 template <typename T> inline void readLittleEndianDynamicBufferOfType(Files::IStreamPtr &pIStream, T* dest, std::size_t numInstances)
 {
+    static_assert(std::is_arithmetic_v<T>, "Buffer element type is not arithmetic");
     pIStream->read((char*)dest, numInstances * sizeof(T));
     if (pIStream->bad())
         throw std::runtime_error("Failed to read little endian dynamic buffer of " + std::to_string(numInstances) + " instances");