[
https://issues.apache.org/jira/browse/IO-77?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12557184#action_12557184
]
Niall Pemberton commented on IO-77:
-----------------------------------
So where are we at with this, I'd like to put it to bed.
I don't want to take out the explict check/throw for nulls since its a PITA for
users to have to look up a line number to find out whats causing the exception.
Consistency is good, but seems that we don't have that - so I suggest we leave
it as it is (throwing IllegalArgumentException) and sort out the consistency in
IO 2.0 - I've created IO-154 to remember to do this.
If this isn't satisfactory then I suggest people vote on this here:
[ ] -1 I don't agree, revert to throwing NullpointerException
[ ] +1 leave it as it is throwing IllegalArgumentException
> Add convenience moveFille() / moveDirectory methods to FileUtils
> ----------------------------------------------------------------
>
> Key: IO-77
> URL: https://issues.apache.org/jira/browse/IO-77
> Project: Commons IO
> Issue Type: Improvement
> Components: Utilities
> Affects Versions: 1.3.2
> Environment: Operating System: other
> Platform: Other
> Reporter: nicolas de loof
> Assignee: Niall Pemberton
> Priority: Minor
> Fix For: 1.4
>
> Attachments: IO-77.patch, patch_io.txt
>
>
> I'm using FileUtils as it partially solves the missing "move" method for File,
> that is so simple to do in unix shell.
> A full implementation in FileUtils may be great :
> static boolean FileUtils.move(File src, File dest)
> throws IOException
> {
> boolean rename = src.renameTo(dest);
> if (!rename)
> {
> copyFile(file, dest);
> file.delete();
> }
> }
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
