Re: [Plib-devel] Re: [Plib-cvs] plib/src/ssg ssgLoadAC.cxx,1.36,1.37

[Bram S. <br...@sa...>]

steve wrote:
> Bram Stolk wrote:
> 
>> Update of /cvsroot/plib/plib/src/ssg
>> In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv8823
>>
>> Modified Files:
>>     ssgLoadAC.cxx Log Message:
>> Avoid append flag. Fix by Mathias
> 
> 
> -  loader_fd = fopen ( filename, "ra" ) ;
> +  loader_fd = fopen ( filename, "r" ) ;
> 
> 
> Wooaaahhhhhh.
> 
> That's not right!
> 
> The 'a' flag only means 'append' if it's the first character
> in the mode string.  In the second position, it should be
> ignored.  On some systems, an 'a' in the second position
> means "ASCII" (as opposed to 'b' for binary).
> 
> I'm sceptical about this patch.

Well... I'll be happy to reverse it, but I tried to find
information about this "ra" mode. I can't find it on the
net, nor on linux ia32, linux ia64, irix, freebsd manpages.
They only speak of the 'b' suffix, which is normally ignored.

Also, I assume ac is an ascii file, and if on those "some
systems" it is read as binary, surely there is no harm in
that? What happens if you read a file as binary, when in
fact it's ascii?

  Bam

> 
> 
> -------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc. Do you grep through log
> files
> for problems?  Stop!  Download the new AJAX search engine that makes
> searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
> http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
> _______________________________________________
> plib-devel mailing list
> pli...@li...
> https://lists.sourceforge.net/lists/listinfo/plib-devel


-- 
Bram Stolk, VR Engineer SARA, Amsterdam.   tel +31 20 592 3000

"Windows is a 32-bit extension to a 16-bit graphical shell for an 8-bit
 operating system originally coded for a 4-bit microprocessor by a 2-bit
 company that can't stand 1 bit of competition."