Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8137046

hrt_assert_is_valid should not be a macro

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Won't Fix
    • Icon: P4 P4
    • 9
    • 9
    • hotspot
    • None
    • gc
    • generic
    • generic

      The macro hrt_assert_is_valid is defined in gc/g1/heapRegionType.hpp.

      Rather than a macro, I suggest it should instead be a private static member function of HeapRegionType, using NOT_DEBUG_RETURN and an ASSERT-only definition in the .cpp file.

            Unassigned Unassigned
            kbarrett Kim Barrett
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: