Skip to content

Commit d54ac08

Browse files
VSadovVSadov
authored
Merge branch 'release/1.10' into fix17745531
2 parents e84e0bd + 0a1d70b commit d54ac08

File tree

17 files changed

+171
-153
lines changed

17 files changed

+171
-153
lines changed

‎lib/Backend/GlobHashTable.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -329,13 +329,13 @@ class ValueHashTable
329329
{
330330
_TYPENAME SListBase<HashBucket>::Iterator iter2(&this2->table[i]);
331331
iter2.Next();
332-
FOREACH_SLISTBASE_ENTRY_EDITING((HashBucket), bucket, &this->table[i], iter)
332+
FOREACH_SLISTBASE_ENTRY_EDITING(HashBucket, bucket, &this->table[i], iter)
333333
{
334334
while (iter2.IsValid() && bucket.value < iter2.Data().value)
335335
{
336336
HashBucket * newBucket = iter.InsertNodeBefore(this->alloc);
337337
newBucket->value = iter2.Data().value;
338-
newBucket->element = fn(null, iter2.Data().element);
338+
newBucket->element = fn(nullptr, iter2.Data().element);
339339
iter2.Next();
340340
}
341341

@@ -354,7 +354,7 @@ class ValueHashTable
354354
{
355355
HashBucket * newBucket = iter.InsertNodeBefore(this->alloc);
356356
newBucket->value = iter2.Data().value;
357-
newBucket->element = fn(null, iter2.Data().element);
357+
newBucket->element = fn(nullptr, iter2.Data().element);
358358
iter2.Next();
359359
}
360360
}

‎lib/Common/CommonMin.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,8 @@ using namespace Memory;
5353
#include "DataStructures/SList.h"
5454
#include "DataStructures/DList.h"
5555
#include "DataStructures/KeyValuePair.h"
56-
#include "DataStructures/BaseDictionary.h"
5756
#include "DataStructures/DictionaryEntry.h"
57+
#include "DataStructures/BaseDictionary.h"
5858
#include "DataStructures/ClusterList.h"
5959

6060
// === Configurations Header ===

‎lib/Common/DataStructures/ClusterList.h

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,8 @@ class ClusterList
4444
, consolidated(true)
4545
#endif
4646
{
47-
list = AllocatorNewArrayLeaf(TAllocator, this->alloc, indexType, maxCount);
48-
for (indextype i = 0; i < maxIndex; i++)
47+
list = AllocatorNewArrayLeaf(TAllocator, this->alloc, indexType, maxIndex);
48+
for (indexType i = 0; i < maxIndex; i++)
4949
{
5050
list[i] = i;
5151
}
@@ -68,7 +68,7 @@ class ClusterList
6868
{
6969
if (this->list != nullptr)
7070
{
71-
AllocatorDeleteArrayLeaf(TAllocator, this->alloc, maxCount, this->list);
71+
AllocatorDeleteArrayLeaf(TAllocator, this->alloc, maxIndex, this->list);
7272
this->list = nullptr;
7373
}
7474
}
@@ -86,7 +86,7 @@ class ClusterList
8686
// Reset the list; useful if we're re-using the data structure
8787
void Reset()
8888
{
89-
for (indextype i = 0; i < maxIndex; i++)
89+
for (indexType i = 0; i < maxIndex; i++)
9090
{
9191
list[i] = i;
9292
}
@@ -313,7 +313,7 @@ class SegmentClusterList
313313
{
314314
if (backingStore[i] != nullptr)
315315
{
316-
AllocatorDeleteArrayLeaf(TAllocator, alloc, numPerSegment, backingstore[i]);
316+
AllocatorDeleteArrayLeaf(TAllocator, alloc, numPerSegment, backingStore[i]);
317317
backingStore[i] = nullptr;
318318
}
319319
}

‎lib/Common/DataStructures/EvalMapString.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ namespace Js
3434
EvalMapStringInternal& operator=(void * str)
3535
{
3636
Assert(str == null);
37-
memset(this, 0, sizeof(EvalMapString));
37+
memset(this, 0, sizeof(*this));
3838
return (*this);
3939
}
4040

‎lib/Common/DataStructures/List.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -341,7 +341,7 @@ namespace JsUtil
341341
template <typename TConditionalFunction>
342342
bool Last(TConditionalFunction function, T& outElement)
343343
{
344-
for (int i = count - 1; i >= 0; --i)
344+
for (int i = this->count - 1; i >= 0; --i)
345345
{
346346
if (function(this->buffer[i]))
347347
{
@@ -420,7 +420,7 @@ namespace JsUtil
420420
JsUtil::ExternalApi::RaiseOnIntOverflow();
421421
}
422422

423-
js_memcpy_s(buffer + this->count, availableByteSpace, items, givenBufferSize);
423+
js_memcpy_s(this->buffer + this->count, availableByteSpace, items, givenBufferSize);
424424
this->count = requiredSize;
425425

426426
return requiredSize; //Returns count

‎lib/Common/Memory/AutoAllocatorObjectPtr.h

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -61,10 +61,10 @@ class AutoAllocatorArrayPtr : public BasePtr<T>
6161
private:
6262
void Clear()
6363
{
64-
if (ptr != nullptr)
64+
if (this->ptr != nullptr)
6565
{
66-
DeleteArray<TAllocator>(m_allocator, m_elementCount, ptr);
67-
ptr = nullptr;
66+
DeleteArray<TAllocator>(m_allocator, this->m_elementCount, this->ptr);
67+
this->ptr = nullptr;
6868
}
6969
}
7070
};
@@ -98,14 +98,14 @@ class AutoAllocatorObjectArrayPtr : public AutoAllocatorArrayPtr<T*, ArrayAlloca
9898
private:
9999
void Clear()
100100
{
101-
if (ptr != nullptr)
101+
if (this->ptr != nullptr)
102102
{
103-
for (size_t i = 0; i < m_elementCount; i++)
103+
for (size_t i = 0; i < this->m_elementCount; i++)
104104
{
105-
if (ptr[i] != nullptr)
105+
if (this->ptr[i] != nullptr)
106106
{
107-
DeleteObject<TAllocator>(m_allocator, ptr[i]);
108-
ptr[i] = nullptr;
107+
DeleteObject<TAllocator>(this->m_allocator, this->ptr[i]);
108+
this->ptr[i] = nullptr;
109109
}
110110
}
111111
}

‎lib/Common/Memory/AutoPtr.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -76,18 +76,18 @@ class AutoArrayAndItemsPtr : public AutoArrayPtr<T>
7676
private:
7777
void Clear()
7878
{
79-
if (ptr != nullptr){
79+
if (this->ptr != nullptr){
8080
for (size_t i = 0; i < this->m_elementCount; i++)
8181
{
82-
if (ptr[i] != nullptr)
82+
if (this->ptr[i] != nullptr)
8383
{
84-
ptr[i]->CleanUp();
85-
ptr[i] = nullptr;
84+
this->ptr[i]->CleanUp();
85+
this->ptr[i] = nullptr;
8686
}
8787
}
8888

89-
HeapDeleteArray(m_elementCount, ptr);
90-
ptr = nullptr;
89+
HeapDeleteArray(this->m_elementCount, this->ptr);
90+
this->ptr = nullptr;
9191
}
9292
}
9393
};

‎lib/Common/PlatformAgnostic/Arrays.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ namespace Arrays
3131
bool GetLocaleSeparator(char16* szSeparator, uint32* sepOutLength, uint32 sepBufferSize);
3232

3333
template <uint32 sepBufferSize>
34-
inline bool GetLocaleSeparator(char16(&szSepatator)[sepBufferSize], uint32 *sepOutLength)
34+
inline bool GetLocaleSeparator(char16(&szSeparator)[sepBufferSize], uint32 *sepOutLength)
3535
{
3636
return GetLocaleSeparator(szSeparator, sepOutLength, sepBufferSize);
3737
}

0 commit comments

Comments
 (0)