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

Constructor in Records with varargs not found #2640

Closed
brunoborges opened this issue Aug 22, 2022 · 7 comments
Closed

Constructor in Records with varargs not found #2640

brunoborges opened this issue Aug 22, 2022 · 7 comments

Comments

@brunoborges
Copy link

Environment
  • JDK version: 17
Steps To Reproduce
  1. Create a Record with last argument being varargs
  2. Attempt to instantiate the record using varargs, from another class.
Current Result

image

Expected Result

Constructor should be found.

Additional Informations

If the record is declared within the same class instantiating it, the extension works fine.

@snjeza
Copy link
Contributor

snjeza commented Aug 22, 2022

This is an upstream issue. It can be reproduced in Eclipse.

@rgrunber
Copy link
Member

rgrunber commented Aug 23, 2022

Reproduced upstream with Eclipse :
Version: 2022-09 (4.25)
Build id: I20220823-0600

Even stranger is I don't see the error reported in the problems view though it is clearly shown in the editor 😐

@snjeza , is there a known bug for this we can reference (even old one in bugs.eclipse.org) ? If none, we should probably file a new one.

@rgrunber rgrunber added the bug label Aug 23, 2022
@rgrunber rgrunber changed the title [Bug] Constructor in Records with varargs not found Constructor in Records with varargs not found Aug 23, 2022
@snjeza
Copy link
Contributor

snjeza commented Aug 23, 2022

, is there a known bug for this we can reference (even old one in bugs.eclipse.org) ? If none, we should probably file a new one.

@rgrunber I can't find it.

@rgrunber
Copy link
Member

Just noticed that the code runs just fine for me, if the above is the only error 😐 . @brunoborges , can you confirm that ? I've included it in the report upstream.

@brunoborges
Copy link
Author

Yeah, code works.

@snjeza
Copy link
Contributor

snjeza commented Dec 26, 2022

A related PR - eclipse-jdt/eclipse.jdt.core#618

@snjeza
Copy link
Contributor

snjeza commented Mar 15, 2023

Fixed with eclipse-jdtls/eclipse.jdt.ls#2538

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants