[ 
https://issues.apache.org/jira/browse/VFS-519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14217385#comment-14217385
 ] 

Bernd Eckenfels edited comment on VFS-519 at 11/19/14 4:47 AM:
---------------------------------------------------------------

Thanks for the patch, applied it with
http://svn.apache.org/r1640485
(I am not sure about the `{@code null}` lets see of somebody comments on it :) 

The unit test is (cleaned up) here: http://svn.apache.org/r1640486




was (Author: b.eckenfels):
Thanks for the patch, applied it with
http://svn.apache.org/r1640485

Will apply the additional unit test after cleanup.

(I am not sure about the `{@code null}` lets see of somebody comments on it :) 


> Javadoc is wrong in some places
> -------------------------------
>
>                 Key: VFS-519
>                 URL: https://issues.apache.org/jira/browse/VFS-519
>             Project: Commons VFS
>          Issue Type: Bug
>            Reporter: Roland Illig
>            Assignee: Bernd Eckenfels
>            Priority: Minor
>              Labels: documentation, patch
>             Fix For: 2.1
>
>         Attachments: VFS-519.patch
>
>
> * {{FileSystemManager.resolveFile(File, String)}} says that the first 
> parameter may be {{null}}. When I pass {{null}}, a {{NullPointerException}} 
> is thrown.
> * {{FileSystem.getLastModTimeAccuracy}}: the {{\@return}} clause is not 
> grammatically correct.
> * {{SftpFileSystemConfigBuilder.setCompression}}: mentions the "jsch 
> documentation"; first, jsch does not have documentation, second, the 
> reference should be a little more specific.
> * {{SftpFileSystemConfigBuilder.getCompression}}: should be aligned with the 
> {{setCompression}} documentation: when I pass {{"zlib,none"}}, it is not 
> returned as "the compression algorithm", but as a comma-separated list of 
> compression algorithms.
> * {{SftpFileSystemConfigBuilder.setTimeout}} does not mention the time unit. 
> It is milliseconds.
> * {{FtpFileSystemConfigBuilder.setTimeout}} does not mention the time unit. 
> Same for {{getSoTimeout}}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to