commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jukka Zitting <>
Subject Re: [PATCH]Commons IO New Functionality, Minor Patches
Date Thu, 29 Oct 2009 14:29:39 GMT

On Wed, Oct 28, 2009 at 4:42 AM, David Armstrong
<> wrote:
> The new functionality is a class that copies files, called FileCopier
> and some helper classes. This class goes beyond the functionality of
> the static methods included in the FileUtils class. It has the
> following functionality:

Sounds like an useful addition. Can you file a feature request about
this in and attach the code
there? It would be easier for us to review the code if you could
provide it as a patch against the latest svn trunk. That way it would
be clearer what parts you have changed and how. You can get a nicely
formatted patch with "svn diff" after you've "svn add"ed all the new
files you've created.

> The patches I included are pretty minor. For, I changed
> the access for doCopyFile() from private to protected so that
> FileCopier could make use of its functionality. For, I
> replaced the multiple closeQuietly() methods with one closeQuietly()
> method that takes an object that implements Closeable as its argument.
> If these patches are not acceptable, please let me know.

The latter change sounds like something that could break binary
backwards compatibility.


Jukka Zitting

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message