Redstone1024/Redcraft
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

feat(memory): add CopyConstruct and the Count parameter uses the size_t

Browse Source
This commit is contained in:
_Redstone_c_ 2022-04-30 21:33:18 +08:00
parent 7aadebba70
commit c2c4a03974
2 changed files with 45 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Redcraft.Utility/Source/Private/Testing/MemoryTesting.cpp
View File

@ -146,10 +146,10 @@ NAMESPACE_UNNAMED_BEGIN
struct FTracker struct FTracker
{ {
static int32 Status; static int32 Status;
FTracker() { always_check(Status == 0); Status = -1; } FTracker() { always_check(Status == 0); Status = -1; }
FTracker(const FTracker&) { always_check(Status == 1); Status = -1; } FTracker(const FTracker&) { always_check(Status == 1); Status = -1; }
FTracker(FTracker&&) { always_check(Status == 2); Status = -1; } FTracker(FTracker&&) { always_check(Status == 2); Status = -1; }
~FTracker() { always_check(Status == 3); Status = -1; } ~FTracker() { always_check(Status == 3); Status = -1; }
FTracker& operator=(const FTracker&) { always_check(Status == 4); Status = -1; return *this; } FTracker& operator=(const FTracker&) { always_check(Status == 4); Status = -1; return *this; }
FTracker& operator=(FTracker&&) { always_check(Status == 5); Status = -1; return *this; } FTracker& operator=(FTracker&&) { always_check(Status == 5); Status = -1; return *this; }
friend bool operator==(const FTracker&, const FTracker&) { always_check(Status == 6); Status = -1; return true; } friend bool operator==(const FTracker&, const FTracker&) { always_check(Status == 6); Status = -1; return true; }
@ -173,6 +173,10 @@ void TestMemoryOperator()
Memory::Construct(PtrA, PtrB); Memory::Construct(PtrA, PtrB);
always_check(FTracker::Status == -1); always_check(FTracker::Status == -1);
FTracker::Status = 1;
Memory::CopyConstruct(PtrA, PtrB);
always_check(FTracker::Status == -1);
FTracker::Status = 2; FTracker::Status = 2;
Memory::MoveConstruct(PtrA, PtrB); Memory::MoveConstruct(PtrA, PtrB);
always_check(FTracker::Status == -1); always_check(FTracker::Status == -1);

57
Redcraft.Utility/Source/Public/Memory/MemoryOperator.h
View File

@ -12,9 +12,9 @@ NAMESPACE_MODULE_BEGIN(Utility)
NAMESPACE_BEGIN(Memory) NAMESPACE_BEGIN(Memory)
template <typename ElementType, typename SizeType = int32> template <typename ElementType>
requires (TIsDefaultConstructible<ElementType>::Value || TIsZeroConstructible<ElementType>::Value) requires (TIsDefaultConstructible<ElementType>::Value || TIsZeroConstructible<ElementType>::Value)
FORCEINLINE void DefaultConstruct(ElementType* Address, SizeType Count = 1) FORCEINLINE void DefaultConstruct(ElementType* Address, size_t Count = 1)
{ {
if constexpr (TIsZeroConstructible<ElementType>::Value) if constexpr (TIsZeroConstructible<ElementType>::Value)
{ {
@ -32,9 +32,9 @@ FORCEINLINE void DefaultConstruct(ElementType* Address, SizeType Count = 1)
} }
} }
template <typename DestinationElementType, typename SourceElementType, typename SizeType = int32> template <typename DestinationElementType, typename SourceElementType>
requires (TIsConstructible<DestinationElementType, const SourceElementType&>::Value || TIsBitwiseConstructible<DestinationElementType, const SourceElementType&>::Value) requires (TIsConstructible<DestinationElementType, const SourceElementType&>::Value || TIsBitwiseConstructible<DestinationElementType, const SourceElementType&>::Value)
FORCEINLINE void Construct(DestinationElementType* Destination, const SourceElementType* Source, SizeType Count = 1) FORCEINLINE void Construct(DestinationElementType* Destination, const SourceElementType* Source, size_t Count = 1)
{ {
if constexpr (TIsBitwiseConstructible<DestinationElementType, const SourceElementType>::Value) if constexpr (TIsBitwiseConstructible<DestinationElementType, const SourceElementType>::Value)
{ {
@ -52,9 +52,29 @@ FORCEINLINE void Construct(DestinationElementType* Destination, const SourceElem
} }
} }
template <typename ElementType, typename SizeType = int32> template <typename ElementType>
requires (TIsCopyConstructible<ElementType>::Value)
FORCEINLINE void CopyConstruct(ElementType* Destination, const ElementType* Source, size_t Count = 1)
{
if constexpr (TIsTriviallyCopyConstructible<ElementType>::Value)
{
Memory::Memcpy(Destination, Source, sizeof(ElementType) * Count);
}
else
{
while (Count)
{
new (Destination) ElementType(*Source);
++(ElementType*&)Destination;
++Source;
--Count;
}
}
}
template <typename ElementType>
requires (TIsMoveConstructible<ElementType>::Value) requires (TIsMoveConstructible<ElementType>::Value)
FORCEINLINE void MoveConstruct(ElementType* Destination, ElementType* Source, SizeType Count = 1) FORCEINLINE void MoveConstruct(ElementType* Destination, ElementType* Source, size_t Count = 1)
{ {
if constexpr (TIsTriviallyMoveConstructible<ElementType>::Value) if constexpr (TIsTriviallyMoveConstructible<ElementType>::Value)
{ {
@ -72,9 +92,9 @@ FORCEINLINE void MoveConstruct(ElementType* Destination, ElementType* Source, Si
} }
} }
template <typename DestinationElementType, typename SourceElementType, typename SizeType = int32> template <typename DestinationElementType, typename SourceElementType>
requires ((TIsConstructible<DestinationElementType, SourceElementType&&>::Value && TIsDestructible<SourceElementType>::Value) || TIsBitwiseRelocatable<DestinationElementType, SourceElementType>::Value) requires ((TIsConstructible<DestinationElementType, SourceElementType&&>::Value && TIsDestructible<SourceElementType>::Value) || TIsBitwiseRelocatable<DestinationElementType, SourceElementType>::Value)
FORCEINLINE void RelocateConstruct(DestinationElementType* Destination, SourceElementType* Source, SizeType Count = 1) FORCEINLINE void RelocateConstruct(DestinationElementType* Destination, SourceElementType* Source, size_t Count = 1)
{ {
if constexpr (TIsBitwiseRelocatable<DestinationElementType, SourceElementType>::Value) if constexpr (TIsBitwiseRelocatable<DestinationElementType, SourceElementType>::Value)
{ {
@ -94,9 +114,9 @@ FORCEINLINE void RelocateConstruct(DestinationElementType* Destination, SourceEl
} }
} }
template <typename ElementType, typename SizeType = int32> template <typename ElementType>
requires (TIsDestructible<ElementType>::Value) requires (TIsDestructible<ElementType>::Value)
FORCEINLINE void Destruct(ElementType* Element, SizeType Count = 1) FORCEINLINE void Destruct(ElementType* Element, size_t Count = 1)
{ {
if constexpr (!TIsTriviallyDestructible<ElementType>::Value) if constexpr (!TIsTriviallyDestructible<ElementType>::Value)
{ {
@ -111,9 +131,9 @@ FORCEINLINE void Destruct(ElementType* Element, SizeType Count = 1)
} }
} }
template <typename ElementType, typename SizeType = int32> template <typename ElementType>
requires (TIsCopyAssignable<ElementType>::Value) requires (TIsCopyAssignable<ElementType>::Value)
FORCEINLINE void CopyAssign(ElementType* Destination, const ElementType* Source, SizeType Count = 1) FORCEINLINE void CopyAssign(ElementType* Destination, const ElementType* Source, size_t Count = 1)
{ {
if constexpr (TIsTriviallyCopyAssignable<ElementType>::Value) if constexpr (TIsTriviallyCopyAssignable<ElementType>::Value)
{ {
@ -131,9 +151,9 @@ FORCEINLINE void CopyAssign(ElementType* Destination, const ElementType* Source,
} }
} }
template <typename ElementType, typename SizeType = int32> template <typename ElementType>
requires (TIsMoveAssignable<ElementType>::Value) requires (TIsMoveAssignable<ElementType>::Value)
FORCEINLINE void MoveAssign(ElementType* Destination, ElementType* Source, SizeType Count = 1) FORCEINLINE void MoveAssign(ElementType* Destination, ElementType* Source, size_t Count = 1)
{ {
if constexpr (TIsTriviallyCopyConstructible<ElementType>::Value) if constexpr (TIsTriviallyCopyConstructible<ElementType>::Value)
{ {
@ -151,9 +171,9 @@ FORCEINLINE void MoveAssign(ElementType* Destination, ElementType* Source, SizeT
} }
} }
template <typename ElementType, typename SizeType = int32> template <typename ElementType>
requires (CEqualityComparable<ElementType> || TIsBitwiseComparable<ElementType>::Value) requires (CEqualityComparable<ElementType> || TIsBitwiseComparable<ElementType>::Value)
FORCEINLINE bool Compare(const ElementType* LHS, const ElementType* RHS, SizeType Count = 1) FORCEINLINE bool Compare(const ElementType* LHS, const ElementType* RHS, size_t Count = 1)
{ {
if constexpr (TIsBitwiseComparable<ElementType>::Value) if constexpr (TIsBitwiseComparable<ElementType>::Value)
{ {
@ -163,10 +183,7 @@ FORCEINLINE bool Compare(const ElementType* LHS, const ElementType* RHS, SizeTyp
{ {
while (Count) while (Count)
{ {
if (!(*LHS == *RHS)) if (!(*LHS == *RHS)) return false;
{
return false;
}
++LHS; ++LHS;
++RHS; ++RHS;