Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introducing some mistakes #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Introducing some mistakes #3

wants to merge 2 commits into from

Conversation

lahodaj
Copy link
Owner

@lahodaj lahodaj commented Feb 3, 2020

No description provided.

@@ -3020,6 +3020,7 @@ public void visitLambda(final JCLambda that) {
if (needsRecovery) {
attribTree(that, env, recoveryInfo);
}
return ; //wrong

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jackpot:
warning: The 'return' statement in the 'finally' block discards unhandled exceptions

@@ -2,7 +2,7 @@
* @test /nodynamiccopyright/
* @bug 8231827
* @summary Clashing bindings are reported correctly
* @compile/fail/ref=BindingsExistTest.out -XDrawDiagnostics --enable-preview -source ${jdk.version} BindingsExistTest.java
* @compile/fail/ref=BindingsExistTest.out -XDrawDiagnostics --enable-preview -source 15 BindingsExistTest.java

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jackpot:
warning: Hardcoded source version, should use ${jdk.version}

@@ -5,7 +5,7 @@
* @library /tools/javac/lib
* @modules jdk.compiler
* @build JavacTestingAbstractProcessor TestMultipleErrors
* @compile/fail/ref=TestMultipleErrors.out -XDrawDiagnostics -processor TestMultipleErrors TestMultipleErrors.java
* @compile/fail -XDrawDiagnostics -processor TestMultipleErrors TestMultipleErrors.java

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jackpot:
warning: Reference output missing

lahodaj pushed a commit that referenced this pull request Nov 3, 2021
Improvements to JDK-8224922
lahodaj pushed a commit that referenced this pull request Mar 13, 2023
lahodaj pushed a commit that referenced this pull request Mar 22, 2024
Fixing positions when transforming javadoc text with replacement characters.
lahodaj pushed a commit that referenced this pull request Sep 26, 2024
Reviewed-by: honkar, prr
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants