garydgregory commented on pull request #215: URL: https://github.com/apache/commons-io/pull/215#issuecomment-922598115
@shollander I modified this PR 215 for git master with: - Do not write the console. - Simplify cleaner check support. - Simplify exception handling. - Don't initialize instance variables to their default values. - Un-nest else clause. - Declaring interfaces as static is redundant. - Use EOF constant instead of magic number. - Don't throw RuntimeException, use IllegalStateException. - Pacakge private class does not need public method. - Use final. - Improve Javadocs. - Javadoc sentences should end in a period. - Javadoc do not need first sentence to be in an HTML p element. - No need to use `this.` when you do not need to. - In-line some single use variables. - Sort members. Please check it out. Feel free to rebase this PR on master or provide a new PR if you need more for your use case. TY! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
