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)

vsiferror.diff (7.5 KB ) - added by dron 15 years ago.
Patch to add VSIFError() wrapper and appropriate virtual filesystem implementation

Download all attachments as: .zip

Change History (6)

by dron, 15 years ago

Attachment: vsiferror.diff added

Patch to add VSIFError() wrapper and appropriate virtual filesystem implementation

comment:1 by dron, 15 years ago

Cc: warmerdam added

comment:2 by warmerdam, 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 Jukka Rahkonen, 8 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:4 by Even Rouault, 8 years ago

Milestone: 1.8.1

Removing obsolete milestone

comment:5 by Jukka Rahkonen, 8 years ago

Resolution: wontfix
Status: newclosed

If it was valid there was plenty of time to react, closing now.

Note: See TracTickets for help on using tickets.