commit 68c676f8e27234251d8acdcaa4d8251ccbf6cdaf
author Adam Sawicki <adam.sawicki@amd.com> Fri Jun 25 17:50:40 2021 +0200
committer Adam Sawicki <adam.sawicki@amd.com> Fri Jun 25 17:50:40 2021 +0200
tree b1b0ea621dd19e2ad570a9cb4cea38ec90d8ac3a
parent 18eab0d2266e01653420d91bbb296d264062b3ac

Used range-based for loops wherever I could

src/D3D12MemAlloc.cpp

diff --git a/src/D3D12MemAlloc.cpp b/src/D3D12MemAlloc.cpp
index f38e597..54b8522 100644
--- a/src/D3D12MemAlloc.cpp
+++ b/src/D3D12MemAlloc.cpp
@@ -1762,6 +1762,9 @@
     const_iterator cbegin() const { return const_iterator(this, Front()); }

     const_iterator cend() const { return const_iterator(this, NULL); }

 

+    const_iterator begin() const { return cbegin(); }

+    const_iterator end() const { return cend(); }

+

     void clear() { Clear(); }

     void push_back(const T& value) { PushBack(value); }

     void erase(iterator it) { Remove(it.m_pItem); }

@@ -3141,12 +3144,8 @@
     // True if previous visited suballocation was free.

     bool prevFree = false;

 

-    for(SuballocationList::const_iterator suballocItem = m_Suballocations.cbegin();

-        suballocItem != m_Suballocations.cend();

-        ++suballocItem)

+    for(const auto& subAlloc : m_Suballocations)

     {

-        const Suballocation& subAlloc = *suballocItem;

-

         // Actual offset of this suballocation doesn't match expected one.

         D3D12MA_VALIDATE(subAlloc.offset == calculatedOffset);

 

@@ -3233,11 +3232,8 @@
 

 void BlockMetadata_Generic::GetAllocationInfo(UINT64 offset, VIRTUAL_ALLOCATION_INFO& outInfo) const

 {

-    for(SuballocationList::const_iterator suballocItem = m_Suballocations.cbegin();

-        suballocItem != m_Suballocations.cend();

-        ++suballocItem)

+    for(const auto& suballoc : m_Suballocations)

     {

-        const Suballocation& suballoc = *suballocItem;

         if(suballoc.offset == offset)

         {

             outInfo.size = suballoc.size;

@@ -3590,11 +3586,8 @@
 

 void BlockMetadata_Generic::SetAllocationUserData(UINT64 offset, void* userData)

 {

-    for(SuballocationList::iterator suballocItem = m_Suballocations.begin();

-        suballocItem != m_Suballocations.end();

-        ++suballocItem)

+    for(auto& suballoc : m_Suballocations)

     {

-        Suballocation& suballoc = *suballocItem;

         if(suballoc.offset == offset)

         {

             suballoc.userData = userData;

@@ -3620,11 +3613,8 @@
     outInfo.UnusedRangeSizeMin = UINT64_MAX;

     outInfo.UnusedRangeSizeMax = 0;

 

-    for(SuballocationList::const_iterator suballocItem = m_Suballocations.cbegin();

-        suballocItem != m_Suballocations.cend();

-        ++suballocItem)

+    for(const auto& suballoc : m_Suballocations)

     {

-        const Suballocation& suballoc = *suballocItem;

         if(suballoc.type == SUBALLOCATION_TYPE_FREE)

         {

             outInfo.UnusedRangeSizeMin = D3D12MA_MIN(suballoc.size, outInfo.UnusedRangeSizeMin);

@@ -3651,11 +3641,8 @@
     json.WriteNumber(m_FreeCount);

     json.WriteString(L"Suballocations");

     json.BeginArray();

-    for(SuballocationList::const_iterator suballocItem = m_Suballocations.cbegin();

-        suballocItem != m_Suballocations.cend();

-        ++suballocItem)

+    for(const auto& suballoc : m_Suballocations)

     {

-        const Suballocation& suballoc = *suballocItem;

         json.BeginObject(true);

         json.WriteString(L"Offset");

         json.WriteNumber(suballoc.offset);