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

Change various JVM enums like LinkInfo::AccessCheck and Klass::DefaultsLookupMode to enum class

    XMLWordPrintable

Details

    • Enhancement
    • Status: Resolved
    • P4
    • Resolution: Fixed
    • 15
    • 16
    • hotspot
    • b11

    Description

      The following enums:

      linkResolver.hpp - class LinkInfo

        enum AccessCheck {
          needs_access_check,
          skip_access_check
        };

      klass.hpp - class Klass

        enum DefaultsLookupMode { find_defaults, skip_defaults };
        enum OverpassLookupMode { find_overpass, skip_overpass };
        enum StaticLookupMode { find_static, skip_static };
        enum PrivateLookupMode { find_private, skip_private };

      Should be changed to C++11 enum class to provide better type safety. Current implementation allows for the enums to be incorrectly compared to boolean true or false.

      Attachments

        Issue Links

          is blocked by

          JEP - Java Enhancement Proposal JDK-8208089 JEP 347: Enable C++14 Language Features

          • P3 - Major loss of function.
          • Closed
          relates to

          Bug - A problem which impairs or prevents the functions of the product. JDK-8244090 public lookup should find public members of public exported types

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Enhancement - null JDK-8252464 Convert SystemDictionary::WKID to enum class

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Enhancement - null JDK-8252526 Remove excessive inclusion of jvmti.h and jvmtiExport.hpp

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Enhancement - null JDK-8253402 Convert vmSymbols::SID to enum class

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Activity

            People

              lfoltan Lois Foltan
              lfoltan Lois Foltan
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: