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

C++11 requires a space between literal and identifier

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: P4
    • Resolution: Fixed
    • Affects Version/s: 9
    • Fix Version/s: 9
    • Component/s: hotspot
    • Labels:
    • Subcomponent:
    • Resolved In Build:
      b79

      Backports

        Description

        "8081202 C++11 requires a space between literal and identifier", see

        http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-May/018665.html

        fixed a number of places in hotspot coding with missing spaces between literals and identifiers.

        When experimenting with Visual Studio 2015 (which refuses compiling such code) , I noticed that this fix is missing here (introduced by "8087333: Optionally Pre-Generate the HotSpot Template Interpreter"):

        hotspot/src/share/vm/interpreter/interpreterRuntime.cpp

        The following small diff (contributed by "matthias.baesken@sap.com") is fixing the remaining issue.


        1306c1306
        < tty->print_cr("argument handler #%d at "PTR_FORMAT" for fingerprint " UINT64_FORMAT,
        ---
        > tty->print_cr("argument handler #%d at " PTR_FORMAT " for fingerprint " UINT64_FORMAT,
        1316c1316
        < tty->print_cr("duplicate argument handler #%d for fingerprint " UINT64_FORMAT "(old: "PTR_FORMAT", new : "PTR_FORMAT")",
        ---
        > tty->print_cr("duplicate argument handler #%d for fingerprint " UINT64_FORMAT "(old: " PTR_FORMAT ", new : " PTR_FORMAT ")",

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  simonis Volker Simonis
                  Reporter:
                  simonis Volker Simonis
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: