Re: ofxdump/ofxdump.cpp 1.26: Removing sys/stat.h

On Tuesday 01 August 2006 23:54, Ace Jones wrote:
> It looks like I was mistaken about it crashing.  The problem is that it
> gives this spew:
>
> LibOFX INFO: libofx_proc_file(): File format not specified, autodecting...
> (Above message occured on Line 4294967295, Column 4294967295)
> LibOFX ERROR: libofx_detect_file_type():Unable to open the input file x
> (Above message occured on Line 4294967295, Column 4294967295)
> LibOFX INFO: libofx_proc_file(): Detected file format: UNKNOWN (File format
> couldn't be sucessfully identified) (Above message occured on Line
> 4294967295, Column 4294967295)
> LibOFX ERROR: libofx_proc_file(): Detected file format not yet supported ou
> couldn't detect file format; aborting. (Above message occured on Line
> 4294967295, Column 4294967295)
>
> By inexperienced users, this has been reported as a problem with file
> format.  As in, "Why isn't this file supported?  Why can't my file be
> detected."  This results in extraneous support mails when the program
> could just clearly tell the user what the problem is with a single error
> message.

That's what I meant by "the flow could be improved".  The file should be 
opened by the library, outside the detection routine, and then passed as a 
pointer to the autodetection (or elsewhere if autodetection isn't required).  
In any event, this needen't be done at the executable level.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV