-
Notifications
You must be signed in to change notification settings - Fork 28
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
JXR-135: IT and some refactoring #6
Conversation
Hi Alina, Karl Heinz already picked one of the other PRs for JXR-135. However, the refactoring part is worth adopting. If you can isolate that part, I'm willing to merge those too. |
Ah sorry have realized that... |
Yes, I have seen that another pull request, nice! |
# Conflicts: # maven-jxr/src/main/java/org/apache/maven/jxr/pacman/JavaFileImpl.java # maven-jxr/src/test/java/org/apache/maven/jxr/pacman/JavaFileImplTest.java
Done: pulled last changes from master, eliminated my partial solution from JavaFileImpl keeping IT for JXR-135 (hope it is useful to see the real output for nested inner classes) and refactoring. |
The provided solution will produce proper names for inner classes of one level, what should cover majority of cases.
Covering all possible levels of inner classes is more complicated, but could be built on top of this work.
P.S. I also did minor cleanup while looking through the files (like introducing diamond operator etc).