[
https://issues.apache.org/jira/browse/IO-205?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sebb updated IO-205:
--------------------
Component/s: Utilities
Fix Version/s: 2.0
> FileUtils.forceMkdir Javadoc is unclear
> ---------------------------------------
>
> Key: IO-205
> URL: https://issues.apache.org/jira/browse/IO-205
> Project: Commons IO
> Issue Type: Bug
> Components: Utilities
> Reporter: Trejkaz
> Fix For: 2.0
>
>
> The Javadoc for FileUtils.forceMkdir doesn't say what is supposed to happen
> if a directory already exists with the same path.
> It does (misleadingly) say that if a file already exists, it fails. I
> interpreted this to mean that a directory would also fail, as it said *file*
> and not *normal file*. However in reality, if the directory already exists,
> the method passes.
> For clarity it should say that it succeeds if the directory already exists,
> and should probably also explicitly say "normal file" instead of just "file",
> as all directories are files.
> Also, internally, I notice it is checking for File.isFile() whereas our
> existing method for implementing the same thing is checking
> !File.isDirectory(). On some filesystems, a file can be both a directory and
> a normal file, and in this situation I would expect the method to pass,
> whereas with Commons' implementation it would fail. Maybe this is a trip-up
> because Sun themselves documented "normal file" as meaning "something which
> is not a directory", which is going to break the moment someone runs
> something on a more exotic filesystem. :-(
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
