[
https://issues.apache.org/jira/browse/KNOX-867?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15850396#comment-15850396
]
Sandeep More commented on KNOX-867:
-----------------------------------
Thanks a lot [~coheigea] ! the patch looks good, I will check it in.
Again thanks a lot for your contribution !
> Fix FindBugs "bad practice" issues
> ----------------------------------
>
> Key: KNOX-867
> URL: https://issues.apache.org/jira/browse/KNOX-867
> Project: Apache Knox
> Issue Type: Improvement
> Reporter: Colm O hEigeartaigh
> Assignee: Colm O hEigeartaigh
> Fix For: 0.12.0
>
> Attachments: 0001-KNOX-867-Fix-FindBugs-bad-practice-issues.patch
>
>
> Similar to KNOX-792, this task is to fix (some of) the "bad practice" issues
> identified by FindBugs in the Knox code base. The issues fixed in the
> attached patch are:
> a) Not checking the return value of File.mkdirs()
> b) Not implementing Serializable when implementing Comparator
> c) A few performance regressions where keySet is used instead of entrySet
> d) Using platform specific "\n" instead of "%n" in String.format
> e) Not overriding hashCode when overriding equals
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
