[ 
https://issues.apache.org/jira/browse/IO-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12666159#action_12666159
 ] 

Jukka Zitting commented on IO-191:
----------------------------------

Sebb, you're right, the collection field should be volatile. My assumption (not 
true for AndFileFilter) was that the collection itself is immutable, so partial 
updates would not have been a problem.

Anyway, my objection on grounds of thread safety is moot since the 
AndFileFilter is not (and probably does not need to be) thread-safe. So making 
the field final and using Collection.clear() is fine by me.

> Possible improvements using static analysis.
> --------------------------------------------
>
>                 Key: IO-191
>                 URL: https://issues.apache.org/jira/browse/IO-191
>             Project: Commons IO
>          Issue Type: Improvement
>            Reporter: Peter Lawrey
>            Priority: Trivial
>         Attachments: commons-io-static-analysis.patch
>
>   Original Estimate: 3h
>  Remaining Estimate: 3h
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to