JDK-8252120 : compiler/oracle/TestCompileCommand.java misspells "occured"
  • Type: Bug
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 9,11,15,16
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2020-08-20
  • Updated: 2020-12-16
  • Resolved: 2020-08-21
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 11 JDK 15 JDK 16
11.0.11-oracleFixed 15.0.2Fixed 16 b13Fixed
Related Reports
Relates :  
JDK-8135068 - Extract methodmatchers from CompilerOracle
Relates :  
JDK-8069389 - CompilerOracle prefix wildcarding is broken for long strings
Description
JDK-8069389 introduced the new test with "occured" typo. That was not accidental, as before JDK-8135068 that typo was in VM code too:
https://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/e3201914b83b#l5.945

Not after, though:
http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/e3201914b83b#l5.757

Some tests were updated within that change:
http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/e3201914b83b#l13.39

...but TestCompileCommand.java was not.
Comments
Fix Request (15u) This keeps tests consistent across 11u, 15u, 16. Patch applies cleanly, affected test passes.
09-09-2020
Fix Request (11u) This backport enables the effectively disabled assert. Patch applies cleanly to 11u, affected test still passes.
25-08-2020
URL: https://hg.openjdk.java.net/jdk/jdk/rev/ca14fc0f7819 User: shade Date: 2020-08-21 07:38:40 +0000
21-08-2020
I wood szupport zat.
21-08-2020
Maybe. As I look over the source code occurences, there are not so many of them, and they are mostly in comments: $ ack occured src/ | wc -l 93 Perhaphs a bigger spell-checking ecersize is in order!
21-08-2020
Sorry, I was (implicitly) wondering if we should fix all the other places as well, but perhaps as a separate bug?
21-08-2020
Mikael, your point being? $ ack "An error occured during parsing" src/ | wc -l 0
21-08-2020
$ grep -r occured . | wc -l 726
20-08-2020