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

--permit-illegal-access warning should respect -XX:-PrintWarnings

    Details

    • Type: Bug
    • Status: Closed
    • Priority: P3
    • Resolution: Not an Issue
    • Affects Version/s: 9
    • Fix Version/s: None
    • Component/s: core-libs

      Description

      Introduced in 9-b163 as part of:
      JDK-8174823 Module system implementation refresh (3/2017)

      Usage of the new option ( --permit-illegal-access ) prints a warning to stderr. While this is intentional behavior, the warning should respect the -XX:-PrintWarnings option as do other warning messages (like the new CMS deprecation warning in 9).

      I understand the desire to make user's aware, but if they go to lengths to intentionally use -XX:-PrintWarnings, then it's "buyer beware" and we should respect that.

        Activity

        Hide
        alanb Alan Bateman added a comment -
        The warning is deliberately not suppressible.

        Here is the proposal:
         http://mail.openjdk.java.net/pipermail/jigsaw-dev/2017-March/011763.html
        Show
        alanb Alan Bateman added a comment - The warning is deliberately not suppressible. Here is the proposal:   http://mail.openjdk.java.net/pipermail/jigsaw-dev/2017-March/011763.html
        Hide
        alanb Alan Bateman added a comment -
        Hopefully bugs are submitted against the code that is causing the problems. In the meantime then the precise options (--add-exports and --add-opens) can be used to allow the bad code continue to work without any warnings.
        Show
        alanb Alan Bateman added a comment - Hopefully bugs are submitted against the code that is causing the problems. In the meantime then the precise options (--add-exports and --add-opens) can be used to allow the bad code continue to work without any warnings.

          People

          • Assignee:
            Unassigned
            Reporter:
            resii Robert Strout
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: