X-Authentication-Warning: delorie.com: mail set sender to djgpp-workers-bounces using -f Date: Mon, 18 Apr 2005 22:18:11 +0300 From: "Eli Zaretskii" Sender: halo1 AT zahav DOT net DOT il To: djgpp-workers AT delorie DOT com Message-ID: <01c5444b$Blat.v2.4$8250da60@zahav.net.il> Content-Transfer-Encoding: 7BIT Content-Type: text/plain; charset=ISO-8859-1 X-Mailer: emacs 22.0.50 (via feedmail 8 I) and Blat ver 2.4 CC: zack AT codesourcery DOT com, djgpp-workers AT delorie DOT com, gcc-patches AT gcc DOT gnu DOT org In-reply-to: <200504182015.18486.pavenis@latnet.lv> (message from Andris Pavenis on Mon, 18 Apr 2005 20:15:17 +0300) Subject: Re: [libcpp] [PATCH] Handle DOS EOF character for DJGPP References: <200504181846 DOT 38506 DOT pavenis AT latnet DOT lv> <8764yk3vql DOT fsf AT codesourcery DOT com> <200504182015 DOT 18486 DOT pavenis AT latnet DOT lv> Reply-To: djgpp-workers AT delorie DOT com Errors-To: nobody AT delorie DOT com X-Mailing-List: djgpp-workers AT delorie DOT com X-Unsubscribes-To: listserv AT delorie DOT com Precedence: bulk > From: Andris Pavenis > Date: Mon, 18 Apr 2005 20:15:17 +0300 > > On Monday 18 April 2005 19:17, you wrote: > > Andris Pavenis writes: > > > Here is patch for HEAD branch of GCC for handling DOS EOF character > > > (0x1A, ^Z) in libcpp. > > > > > > At least I have met situations when DOS EOF character is written at > > >end of file by some editors which causes compiler to generate error. > > > > This was discussed and rejected some time ago. ^Z can legitimately > > appear in the middle of a file. > > The patch adds handling ^Z for DJGPP only. Right. And a DOS/Windows text file cannot have a ^Z character in its middle, since every program that works with text files, like editors etc., will stop reading at that character. > > The "meat" of this patch does not appear to depend on io.h. Why did > > you add it? > > As I remeber, I had compiler errors while bootstrapping GCC for DJGPP earlier, > but perhaps I must recheck it. The prototype of setmode is in io.h, so I'm guessing that's the reason DJGPP needs it.