[ 
https://issues.apache.org/jira/browse/VFS-337?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb updated VFS-337:
---------------------

    Description: 
The AbstractFileObject constructor accepts FileName as a parameter, but 
actually requires an AbstractFileName.
Anything else will generate a ClassCastException.

The reason for the cast is to allow access to the package-protected method void 
AbstractFileName#setType(FileType type)

Many of the AbstractFileObject methods rely on being able to invoke the setType 
method, so perhaps the parameter should be changed accordingly?
This will change the API and require changes to subclasses.

I'll add Javadoc to document the restriction.

  was:
The AbstractFileName constructor accepts FileName as a parameter, but actually 
requires an AbstractFileName.
Anything else will generate a ClassCastException.

The reason for the cast is to allow access to the package-protected method void 
AbstractFileName#setType(FileType type)

Many of the AbstractFileName methods rely on being able to invoke the setType 
method, so perhaps the parameter should be changed accordingly?
This will change the API and require changes to subclasses.

I'll add Javadoc to document the restriction.

        Summary: AbstractFileObject ctor accepts FileName as a parameter, but 
actually requires AbstractFileName  (was: AbstractFileName ctor accepts 
FileName as a parameter, but actually requires AbstractFileName)

> AbstractFileObject ctor accepts FileName as a parameter, but actually 
> requires AbstractFileName
> -----------------------------------------------------------------------------------------------
>
>                 Key: VFS-337
>                 URL: https://issues.apache.org/jira/browse/VFS-337
>             Project: Commons VFS
>          Issue Type: Bug
>    Affects Versions: 1.0
>            Reporter: Sebb
>
> The AbstractFileObject constructor accepts FileName as a parameter, but 
> actually requires an AbstractFileName.
> Anything else will generate a ClassCastException.
> The reason for the cast is to allow access to the package-protected method 
> void AbstractFileName#setType(FileType type)
> Many of the AbstractFileObject methods rely on being able to invoke the 
> setType method, so perhaps the parameter should be changed accordingly?
> This will change the API and require changes to subclasses.
> I'll add Javadoc to document the restriction.

-- 
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