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

javac.apt's Wrapping ClassLoader NPEs due to a design mistake in j.l.ClassLoader. #683

Closed
lombokissues opened this Issue Jul 14, 2015 · 4 comments

Comments

Projects
None yet
2 participants
@lombokissues
Collaborator

lombokissues commented Jul 14, 2015

Migrated from Google Code (issue 648)

@lombokissues

This comment has been minimized.

Show comment
Hide comment
@lombokissues

lombokissues Jul 14, 2015

Collaborator

👤 reinierz   🕗 Feb 28, 2014 at 02:42 UTC

The super constructor invokes overridable methods, thus a field that couldn't possibly be null, is null.

See https://groups.google.com/forum/﹟!topic/project-lombok/EaLo07CJzIs

Collaborator

lombokissues commented Jul 14, 2015

👤 reinierz   🕗 Feb 28, 2014 at 02:42 UTC

The super constructor invokes overridable methods, thus a field that couldn't possibly be null, is null.

See https://groups.google.com/forum/﹟!topic/project-lombok/EaLo07CJzIs

@lombokissues

This comment has been minimized.

Show comment
Hide comment
@lombokissues

lombokissues Jul 14, 2015

Collaborator

👤 r.spilker   🕗 Mar 01, 2014 at 02:29 UTC

We never could reproduce this problem. However, it seems obvious enough, so, assuming Reinier identified the problem correctly, our last commit ought to fix things.

The edge release has this fix:

https://projectlombok.org/download-edge.html

NB: See lombok intelliJ plugin issue #147: https://code.google.com/p/lombok-intellij-plugin/issues/detail?id=74

We'd love to hear confirmation that the issue is now indeed resolved.

Collaborator

lombokissues commented Jul 14, 2015

👤 r.spilker   🕗 Mar 01, 2014 at 02:29 UTC

We never could reproduce this problem. However, it seems obvious enough, so, assuming Reinier identified the problem correctly, our last commit ought to fix things.

The edge release has this fix:

https://projectlombok.org/download-edge.html

NB: See lombok intelliJ plugin issue #147: https://code.google.com/p/lombok-intellij-plugin/issues/detail?id=74

We'd love to hear confirmation that the issue is now indeed resolved.

@lombokissues lombokissues added the parked label Jul 14, 2015

@lombokissues

This comment has been minimized.

Show comment
Hide comment
@lombokissues

lombokissues Jul 14, 2015

Collaborator

👤 r.spilker   🕗 Mar 10, 2014 at 10:15 UTC

Confirmed. See https://code.google.com/p/lombok-intellij-plugin/issues/detail?id=74

Collaborator

lombokissues commented Jul 14, 2015

👤 r.spilker   🕗 Mar 10, 2014 at 10:15 UTC

Confirmed. See https://code.google.com/p/lombok-intellij-plugin/issues/detail?id=74

@lombokissues

This comment has been minimized.

Show comment
Hide comment
@lombokissues

lombokissues Jul 14, 2015

Collaborator

End of migration

Collaborator

lombokissues commented Jul 14, 2015

End of migration

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