diff options
author | bunnei <bunneidev@gmail.com> | 2020-11-19 05:40:10 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-19 05:40:10 +0100 |
commit | 92344da20ce4801543c1d3148e1f1b62ba162ffb (patch) | |
tree | a5cd5980b1287dfd9b56e6df48cad3790e173284 /src/common/virtual_buffer.h | |
parent | Merge pull request #4866 from Morph1984/mjolnir-p3-prod (diff) | |
parent | virtual_buffer: Add compile-time type-safety guarantees with VirtualBuffer (diff) | |
download | yuzu-92344da20ce4801543c1d3148e1f1b62ba162ffb.tar yuzu-92344da20ce4801543c1d3148e1f1b62ba162ffb.tar.gz yuzu-92344da20ce4801543c1d3148e1f1b62ba162ffb.tar.bz2 yuzu-92344da20ce4801543c1d3148e1f1b62ba162ffb.tar.lz yuzu-92344da20ce4801543c1d3148e1f1b62ba162ffb.tar.xz yuzu-92344da20ce4801543c1d3148e1f1b62ba162ffb.tar.zst yuzu-92344da20ce4801543c1d3148e1f1b62ba162ffb.zip |
Diffstat (limited to '')
-rw-r--r-- | src/common/virtual_buffer.h | 29 |
1 files changed, 24 insertions, 5 deletions
diff --git a/src/common/virtual_buffer.h b/src/common/virtual_buffer.h index 125cb42f0..078e61c77 100644 --- a/src/common/virtual_buffer.h +++ b/src/common/virtual_buffer.h @@ -4,25 +4,44 @@ #pragma once -#include "common/common_funcs.h" +#include <type_traits> +#include <utility> namespace Common { -void* AllocateMemoryPages(std::size_t size); -void FreeMemoryPages(void* base, std::size_t size); +void* AllocateMemoryPages(std::size_t size) noexcept; +void FreeMemoryPages(void* base, std::size_t size) noexcept; template <typename T> -class VirtualBuffer final : NonCopyable { +class VirtualBuffer final { public: + static_assert( + std::is_trivially_constructible_v<T>, + "T must be trivially constructible, as non-trivial constructors will not be executed " + "with the current allocator"); + constexpr VirtualBuffer() = default; explicit VirtualBuffer(std::size_t count) : alloc_size{count * sizeof(T)} { base_ptr = reinterpret_cast<T*>(AllocateMemoryPages(alloc_size)); } - ~VirtualBuffer() { + ~VirtualBuffer() noexcept { FreeMemoryPages(base_ptr, alloc_size); } + VirtualBuffer(const VirtualBuffer&) = delete; + VirtualBuffer& operator=(const VirtualBuffer&) = delete; + + VirtualBuffer(VirtualBuffer&& other) noexcept + : alloc_size{std::exchange(other.alloc_size, 0)}, base_ptr{std::exchange(other.base_ptr), + nullptr} {} + + VirtualBuffer& operator=(VirtualBuffer&& other) noexcept { + alloc_size = std::exchange(other.alloc_size, 0); + base_ptr = std::exchange(other.base_ptr, nullptr); + return *this; + } + void resize(std::size_t count) { FreeMemoryPages(base_ptr, alloc_size); |