From dddfdad6ebb4e7172ff36f5a0db8a5cbf58d45f3 Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Thu, 30 Nov 2017 15:42:00 +0000 Subject: Run clang-tidy modernize-use-auto on //third_party/pdfium See the bugs and cxx post for justification and details: https://groups.google.com/a/chromium.org/forum/#!topic/cxx/RkOHzIK6Tq8 This change was done using clang-tidy as described here: https://chromium.googlesource.com/chromium/src/+/lkcr/docs/clang_tidy.md Bug: chromium:776257 Change-Id: I1f6637cde8b3e41825993a736bed6763dd7beacb Reviewed-on: https://pdfium-review.googlesource.com/19971 Reviewed-by: Lei Zhang Commit-Queue: dsinclair --- .../partition_allocator/partition_alloc.h | 25 +++++++++++----------- 1 file changed, 12 insertions(+), 13 deletions(-) (limited to 'third_party/base/allocator') diff --git a/third_party/base/allocator/partition_allocator/partition_alloc.h b/third_party/base/allocator/partition_allocator/partition_alloc.h index 285f2af5a4..04ae17014a 100644 --- a/third_party/base/allocator/partition_allocator/partition_alloc.h +++ b/third_party/base/allocator/partition_allocator/partition_alloc.h @@ -534,7 +534,7 @@ ALWAYS_INLINE void* PartitionCookieFreePointerAdjust(void* ptr) { ALWAYS_INLINE void PartitionCookieWriteValue(void* ptr) { #if DCHECK_IS_ON() - unsigned char* cookie_ptr = reinterpret_cast(ptr); + auto* cookie_ptr = reinterpret_cast(ptr); for (size_t i = 0; i < kCookieSize; ++i, ++cookie_ptr) *cookie_ptr = kCookieValue[i]; #endif @@ -542,14 +542,14 @@ ALWAYS_INLINE void PartitionCookieWriteValue(void* ptr) { ALWAYS_INLINE void PartitionCookieCheckValue(void* ptr) { #if DCHECK_IS_ON() - unsigned char* cookie_ptr = reinterpret_cast(ptr); + auto* cookie_ptr = reinterpret_cast(ptr); for (size_t i = 0; i < kCookieSize; ++i, ++cookie_ptr) DCHECK(*cookie_ptr == kCookieValue[i]); #endif } ALWAYS_INLINE char* PartitionSuperPageToMetadataArea(char* ptr) { - uintptr_t pointer_as_uint = reinterpret_cast(ptr); + auto pointer_as_uint = reinterpret_cast(ptr); DCHECK(!(pointer_as_uint & kSuperPageOffsetMask)); // The metadata area is exactly one system page (the guard page) into the // super page. @@ -557,8 +557,8 @@ ALWAYS_INLINE char* PartitionSuperPageToMetadataArea(char* ptr) { } ALWAYS_INLINE PartitionPage* PartitionPointerToPageNoAlignmentCheck(void* ptr) { - uintptr_t pointer_as_uint = reinterpret_cast(ptr); - char* super_page_ptr = + auto pointer_as_uint = reinterpret_cast(ptr); + auto* super_page_ptr = reinterpret_cast(pointer_as_uint & kSuperPageBaseMask); uintptr_t partition_page_index = (pointer_as_uint & kSuperPageOffsetMask) >> kPartitionPageShift; @@ -566,7 +566,7 @@ ALWAYS_INLINE PartitionPage* PartitionPointerToPageNoAlignmentCheck(void* ptr) { // the last index is invalid because it is a guard page. DCHECK(partition_page_index); DCHECK(partition_page_index < kNumPartitionPagesPerSuperPage - 1); - PartitionPage* page = reinterpret_cast( + auto* page = reinterpret_cast( PartitionSuperPageToMetadataArea(super_page_ptr) + (partition_page_index << kPageMetadataShift)); // Partition pages in the same slot span can share the same page object. @@ -578,7 +578,7 @@ ALWAYS_INLINE PartitionPage* PartitionPointerToPageNoAlignmentCheck(void* ptr) { } ALWAYS_INLINE void* PartitionPageToPointer(const PartitionPage* page) { - uintptr_t pointer_as_uint = reinterpret_cast(page); + auto pointer_as_uint = reinterpret_cast(page); uintptr_t super_page_offset = (pointer_as_uint & kSuperPageOffsetMask); DCHECK(super_page_offset > kSystemPageSize); DCHECK(super_page_offset < kSystemPageSize + (kNumPartitionPagesPerSuperPage * @@ -590,7 +590,7 @@ ALWAYS_INLINE void* PartitionPageToPointer(const PartitionPage* page) { DCHECK(partition_page_index); DCHECK(partition_page_index < kNumPartitionPagesPerSuperPage - 1); uintptr_t super_page_base = (pointer_as_uint & kSuperPageBaseMask); - void* ret = reinterpret_cast( + auto* ret = reinterpret_cast( super_page_base + (partition_page_index << kPartitionPageShift)); return ret; } @@ -641,9 +641,8 @@ ALWAYS_INLINE size_t PartitionPageGetRawSize(PartitionPage* page) { } ALWAYS_INLINE PartitionRootBase* PartitionPageToRoot(PartitionPage* page) { - PartitionSuperPageExtentEntry* extent_entry = - reinterpret_cast( - reinterpret_cast(page) & kSystemPageBaseMask); + auto* extent_entry = reinterpret_cast( + reinterpret_cast(page) & kSystemPageBaseMask); return extent_entry->root; } @@ -687,7 +686,7 @@ ALWAYS_INLINE void* PartitionBucketAlloc(PartitionRootBase* root, slot_size = raw_size; } size_t no_cookie_size = PartitionCookieSizeAdjustSubtract(slot_size); - char* char_ret = static_cast(ret); + auto* char_ret = static_cast(ret); // The value given to the application is actually just after the cookie. ret = char_ret + kCookieSize; memset(ret, kUninitializedByte, no_cookie_size); @@ -737,7 +736,7 @@ ALWAYS_INLINE void PartitionFreeWithPage(void* ptr, PartitionPage* page) { CHECK(ptr != freelist_head); // Catches an immediate double free. // Look for double free one level deeper in debug. DCHECK(!freelist_head || ptr != PartitionFreelistMask(freelist_head->next)); - PartitionFreelistEntry* entry = static_cast(ptr); + auto* entry = static_cast(ptr); entry->next = PartitionFreelistMask(freelist_head); page->freelist_head = entry; --page->num_allocated_slots; -- cgit v1.2.3