Mail Archives: djgpp-workers/2002/05/09/12:55:08
On Thu, May 09, 2002 at 07:11:12PM +0300, pavenis AT lanet DOT lv wrote:
> On 9 May 2002 at 8:42, Zack Weinberg wrote:
> 
> > On Thu, May 09, 2002 at 01:14:53PM +0300, pavenis AT lanet DOT lv wrote:
> > > 
> > > OK. Here it is (tested for DJGPP with 3.1 branch)
> > 
> > O_TEXT, not O_BINARY.  Also, based on other discussion with Eli, the
> > check should apply to all file descriptors, not just stdin, so please
> > move it below the call to open.
> > 
> 
> I don't think so.
> 
> With O_TEXT we will run into another trouble as length of data in memory will 
> be smaller than one in file. As result read_include_file() will complain that
> input file is shorter than expected.
> 
> So unless we want to remove that check, O_BINARY must stay there.
> 
> Current patch changes mode to O_BINARY in rather specific case when
> one is feeding source in gcc from pipe. For example 
> 	cat foo.c | gcc -c -x c -o foo.o -
> In this case file is opened in text mode by default, and I'm getting
> a warning I mentioned above.
Sorry, I misread !isatty as isatty.  This is fine, then.
zw
- Raw text -