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
Eliminate dead code #606
Eliminate dead code #606
Conversation
protected Node findNode(final Session session, final String path) { | ||
try { | ||
// Remove leading and trailing slashes ... | ||
final String relPath = path.replaceAll("^/+", "").replaceAll("/+$", ""); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this can be simplified to just:
session.getNode(path)
No? I think you might have been confused by the findAndCreate logic, which has to build up the whole tree to the object to find out where to start building nodes from.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, commit incoming.
@barmintor, please add the @SInCE annotation noted in the code comments, then feel free to squash the commits: |
simplify node existence checks in AbstractService and delete commented-out code
ignore checkstyle dot files style conformance in UUIDPidMinter
Eliminate dead code Related to: https://www.pivotaltracker.com/story/show/80959252
Part of a review of the codebase to eliminate unused code