Opened 15 years ago
Closed 8 years ago
#2674 closed defect (wontfix)
[PATCH] Add VSIFError() function to virtual filesystem interface
Reported by: | dron | Owned by: | dron |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | default | Version: | svn-trunk |
Severity: | normal | Keywords: | vsi |
Cc: | warmerdam |
Description
Sometimes it is very handy to use a ferror() function along with the feof(). The patch attached adds an appropriate wrapper to the virtual filesystem interface. I will make virtual filesystem a bit more POSIX-compliant.
Attachments (1)
Change History (6)
by , 15 years ago
Attachment: | vsiferror.diff added |
---|
comment:1 by , 15 years ago
Cc: | added |
---|
comment:2 by , 15 years ago
Keywords: | vsi added |
---|---|
Milestone: | → 1.7.0 |
It is not appropriate to make changes to the VSI set of interfaces at this late state for 1.6. Setting the milestone to 1.7. I am reasonably in factor of this change, but it would need to be implemented for all the existing interfaces, and some reasonable implementation provided when it is not implemented as there are several private implementations of the VSI*L interface layer in existance already.
I would think a review of existing file error handling might also be in order.
I think this is RFC-worthy.
comment:3 by , 9 years ago
Summary: | Add VSIFError() function to virtual filesystem interface → [PATCH] Add VSIFError() function to virtual filesystem interface |
---|
VSI has changed in past 6 years. Is this ticket and patch still valid?
comment:5 by , 8 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
If it was valid there was plenty of time to react, closing now.
Patch to add VSIFError() wrapper and appropriate virtual filesystem implementation