diff options
author | Jan Kotas <jkotas@microsoft.com> | 2019-05-14 19:07:53 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-05-14 19:07:53 -0700 |
commit | 718598334310f6ad331ba2893ee4557beffaa74c (patch) | |
tree | 7bc4830b29442e79be150b67dc0f67473e5ba681 /src/gc/gc.cpp | |
parent | 6a85ca560a7dfa109c6f17c0b76d93373dbd5a00 (diff) | |
download | coreclr-718598334310f6ad331ba2893ee4557beffaa74c.tar.gz coreclr-718598334310f6ad331ba2893ee4557beffaa74c.tar.bz2 coreclr-718598334310f6ad331ba2893ee4557beffaa74c.zip |
Fix issues reported by PREfast static analysis tool (#24577)
Diffstat (limited to 'src/gc/gc.cpp')
-rw-r--r-- | src/gc/gc.cpp | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/src/gc/gc.cpp b/src/gc/gc.cpp index e8f2400c1b..c6a858c8fa 100644 --- a/src/gc/gc.cpp +++ b/src/gc/gc.cpp @@ -28077,15 +28077,7 @@ BOOL gc_heap::find_card(uint32_t* card_table, { card_word_value = *(++last_card_word); } while ((last_card_word < &card_table [card_word_end]) && - -#ifdef _MSC_VER - (card_word_value == (1 << card_word_width)-1) -#else - // if left shift count >= width of type, - // gcc reports error. - (card_word_value == ~0u) -#endif // _MSC_VER - ); + (card_word_value == ~0u /* (1 << card_word_width)-1 */)); bit_position = 0; } } while (card_word_value & 1); @@ -30477,7 +30469,7 @@ size_t gc_heap::joined_youngest_desired (size_t new_allocation) uint32_t memory_load = 0; get_memory_info (&memory_load); settings.exit_memory_load = memory_load; - dprintf (2, ("Current emory load: %d", memory_load)); + dprintf (2, ("Current memory load: %d", memory_load)); size_t final_total = trim_youngest_desired (memory_load, total_new_allocation, total_min_allocation); @@ -37418,7 +37410,7 @@ void GCHeap::DiagScanDependentHandles (handle_scan_fn fn, int gen_number, ScanCo // Go through and touch (read) each page straddled by a memory block. void TouchPages(void * pStart, size_t cb) { - const uint32_t pagesize = OS_PAGE_SIZE; + const size_t pagesize = OS_PAGE_SIZE; _ASSERTE(0 == (pagesize & (pagesize-1))); // Must be a power of 2. if (cb) { |