pdinc-oss commented on pull request #89: URL: https://github.com/apache/commons-vfs/pull/89#issuecomment-665666226
> Thinking out loud: why not adding in local filesystem a FileOperation "ToLocalFile", will be backward compatible and provide this feature in a more natural way IMO (caller can test the ToLocalFile.class exists and call it if so, else fallback on downloading the file). While (I think) I understand the OPs intention, the implementation is way off as the PR comments react to. If a File object is desired, it would have to be a subclass that wraps all of the capability of the FileObject, and MUST not assume any specifics (e.g. "file".equals...). The File API is not that hard to implement, it even has well defined alternate values for most of the functions if it cannot be implemented (e.g. public long lastModified() Returns: ... 0L if the file does not exist). ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org