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

Jump to bottom

Inner Java classes are nested recursively #5925

Closed
MariuszCwikla opened this issue Oct 18, 2019 · 1 comment · Fixed by #5926
Closed

Inner Java classes are nested recursively #5925

MariuszCwikla opened this issue Oct 18, 2019 · 1 comment · Fixed by #5926
Milestone
JRuby 9.2.9.0

Comments

@MariuszCwikla
Copy link
Contributor

jruby 9.2.9.0-SNAPSHOT (2.5.3) 2019-10-14 a8807f2 Java HotSpot(TM) 64-Bit Server VM 25.171-b11 on 1.8.0_171-b11 +jit [mswin32-x86_64]

I discovered simple bug while working on another issue #2639. Below is output from JIRB:

Actual Behavior:

irb(main):001:0> Java::JavaAwt::Desktop::Action
=> Java::JavaAwt::Desktop::Action
irb(main):002:0> Java::JavaAwt::Desktop::Action::Action
=> Java::JavaAwt::Desktop::Action
irb(main):003:0> Java::JavaAwt::Desktop::Action::Action::Action
=> Java::JavaAwt::Desktop::Action

Expected Behavior:

2nd and 3rd should result with NameError (there is no class java.awt.Desktop.Action.Action.Action).
Only 1st case is correct.
MariuszCwikla added a commit to MariuszCwikla/jruby that referenced this issue Oct 18, 2019
@MariuszCwikla
fix class nesting jruby#5925
6c9a177
@MariuszCwikla MariuszCwikla mentioned this issue Oct 18, 2019
Merged
@headius
Copy link
Member

headius commented Oct 23, 2019

Well that's a weird one! Thanks for the PR!

headius added a commit that referenced this issue Oct 23, 2019
@headius
Merge pull request #5926 from MariuszCwikla/fix-class-nesting
45f15c6 
fix class nesting #5925
@kares kares added this to the JRuby 9.2.9.0 milestone Oct 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
JRuby 9.2.9.0
Development

Successfully merging a pull request may close this issue.

fix class nesting #5925 MariuszCwikla/jruby
3 participants
@headius @kares @MariuszCwikla