From 05231d8b08f7d473a4c4cf7640227f41de44ac23 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Fri, 20 Jul 2018 21:40:13 -0400 Subject: [PATCH] vfs: Amend constness on pointers in WriteBytes() and WriteArrays() member functions to be const qualified These functions don't modify the data being pointed to, so these can be pointers to const data --- src/core/file_sys/vfs.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/core/file_sys/vfs.h b/src/core/file_sys/vfs.h index a5213e0cc7..edd689c688 100644 --- a/src/core/file_sys/vfs.h +++ b/src/core/file_sys/vfs.h @@ -93,7 +93,7 @@ struct VfsFile : NonCopyable { // Writes an array of type T, size number_elements to offset in file. // Returns the number of bytes (sizeof(T)*number_elements) written successfully. template - size_t WriteArray(T* data, size_t number_elements, size_t offset = 0) { + size_t WriteArray(const T* data, size_t number_elements, size_t offset = 0) { static_assert(std::is_trivially_copyable::value, "Data type must be trivially copyable."); @@ -103,10 +103,10 @@ struct VfsFile : NonCopyable { // Writes size bytes starting at memory location data to offset in file. // Returns the number of bytes written successfully. template - size_t WriteBytes(T* data, size_t size, size_t offset = 0) { + size_t WriteBytes(const T* data, size_t size, size_t offset = 0) { static_assert(std::is_trivially_copyable::value, "Data type must be trivially copyable."); - return Write(reinterpret_cast(data), size, offset); + return Write(reinterpret_cast(data), size, offset); } // Writes one object of type T to offset in file.