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

Fix Exception template to extend from Exception #7390

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

neilcsmith-net
Copy link
Member

Fix Exception template to extend from Exception when no superclass specified in wizard. It's a minor annoyance but odd to have a built-in template create uncompilable code by default.

Theoretically this is a regression, although it seems to have been there since this was last updated for NB 12.3 in #2275 !

@neilcsmith-net neilcsmith-net added Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) Regression This used to work! labels May 21, 2024
@neilcsmith-net neilcsmith-net added this to the NB23 milestone May 21, 2024
Copy link
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

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

Only eyeballed this, but looks sane to me.

@neilcsmith-net neilcsmith-net merged commit 913cc1e into apache:master Jun 3, 2024
37 checks passed
sid-srini added a commit to sid-srini/javavscode that referenced this pull request Aug 27, 2024
In NB23, [patch #7390](apache/netbeans/pull/7390) fixed the regression
where the newly created exception from template did not extend
Exception, thereby failing to compile.

This temporarily backports the fix until NB23 is incorporated.

Signed-off-by: Siddharth Srinivasan <siddharth.srinivasan@oracle.com>
Achal1607 pushed a commit to oracle/javavscode that referenced this pull request Aug 27, 2024
In NB23, [patch #7390](apache/netbeans/pull/7390) fixed the regression
where the newly created exception from template did not extend
Exception, thereby failing to compile.

This temporarily backports the fix until NB23 is incorporated.

Signed-off-by: Siddharth Srinivasan <siddharth.srinivasan@oracle.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Java [ci] enable extra Java tests (java.completion, java.source.base, java.hints, refactoring.java, form) Regression This used to work!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants