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

VerifyError with unnamed pattern variable in instanceof #2025

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

srikanth-sankaran
Copy link
Contributor

@srikanth-sankaran srikanth-sankaran commented Feb 15, 2024

What it does

Author checklist

@srikanth-sankaran srikanth-sankaran self-assigned this Feb 15, 2024
@srikanth-sankaran srikanth-sankaran added this to the 4.31 RC1 milestone Feb 15, 2024
@srikanth-sankaran
Copy link
Contributor Author

@mpalat, @jarthana - This needs to be included for RC1 - this is a regression.

This is exposed by - not caused by - a recent commit #2016

Underlying defective change was introduced via commit b4875fc made for #1517 and merged on 2024-01-29 at 09:51:08

See #2024 (comment) for an explanation.

Copy link
Contributor

@mpalat mpalat left a comment

Choose a reason for hiding this comment

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

+1
please go ahead and merge

@srikanth-sankaran srikanth-sankaran merged commit b05494a into eclipse-jdt:master Feb 15, 2024
9 checks passed
@srikanth-sankaran
Copy link
Contributor Author

Thanks for the review @mpalat ! Could this be picked up for M3 please ?

@mpalat
Copy link
Contributor

mpalat commented Feb 15, 2024

@MohananRahul Could you please respin to include this for M3? this is a regression and fix is straightforward. Thanks!

@srikanth-sankaran srikanth-sankaran deleted the GH2024 branch February 15, 2024 08:58
@srikanth-sankaran srikanth-sankaran modified the milestones: 4.31 RC1, 4.31 M3 Feb 15, 2024
@iloveeclipse
Copy link
Member

@MohananRahul Could you please respin to include this for M3? this is a regression and fix is straightforward. Thanks!

Please don't do that yet, we are waiting for eclipse-platform/eclipse.platform.ui#1686

@iloveeclipse
Copy link
Member

New build is triggered: https://ci.eclipse.org/releng/job/Builds/job/I-build-4.31/137/

@srikanth-sankaran
Copy link
Contributor Author

New build is triggered: https://ci.eclipse.org/releng/job/Builds/job/I-build-4.31/137/

I have verified that this build

Eclipse SDK

Version: 2024-03 (4.31)
Build id: I20240215-0450

contains the fix. The program from #2024 (comment) compiles and runs fine in this build with no verify error.

mickaelistria pushed a commit to mickaelistria/eclipse.jdt.core that referenced this pull request Feb 16, 2024
robstryker pushed a commit to robstryker/eclipse.jdt.core that referenced this pull request Jul 18, 2024
gayanper pushed a commit to gayanper/eclipse.jdt.core that referenced this pull request Sep 7, 2024
snjeza pushed a commit to snjeza/eclipse.jdt.core that referenced this pull request Oct 21, 2024
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.

[Patterns][Unnamed] VerifyError with unnamed pattern variable in instanceof
3 participants