This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: [patch, fortran] PR34324 - Module files on CRLF systems
- From: Daniel Franke <franke dot daniel at gmail dot com>
- To: fortran at gcc dot gnu dot org
- Cc: Tobias Burnus <burnus at net-b dot de>, gcc-patches at gcc dot gnu dot org
- Date: Thu, 13 Dec 2007 21:59:49 +0100
- Subject: Re: [patch, fortran] PR34324 - Module files on CRLF systems
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:subject:date:user-agent:cc:references:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:message-id; bh=QG4ZEzwYbGhUAxWWBtdjayFaVEh3JDDCNLEDp0s4dpA=; b=WdniL8tzEO8XHk4EgK46g45wCH2Fw+BGgg1tys/U/fkwFeg90GvGXD4wBMhAiX/Bd6GdJSAuKMRJH175pf2MObzFPZ3EplltQHCXaEq1dTQ1kYw1oXhDWKVhOAfPGOQcIX2Kqj9DlJYoSA5uil6dS0CcNP1KB4U9s+YhaHKv90g=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:subject:date:user-agent:cc:references:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:message-id; b=o4AVu6eq/z4JiRwYncMWtpj6KeaSzjMHAXLzBRFonyLHpCKWwE1xCTf2e40q0Ecnp7bACbBYqt9qsmJh9M8xegg9GRDyQYkB6nOMLHh4mLuaNgi2oP0ILyYjtGN2uvtaI+YkNUFkpVQXUgVyEyNw3LBYcvtVgYGWKDPUeinBIN8=
- References: <200712132111.17552.franke.daniel@gmail.com> <4761988C.30108@net-b.de>
On Thursday 13 December 2007 21:39:40 Tobias Burnus wrote:
> > Up to now, the module parser does not like '\r' characters in the input
> > stream and aborts at such. The fix is to ignore these characters, as done
> > in scanner.c (load_line).
>
> I wonder why this error did not show up before.
It seems that user interaction is needed. Either by 'unix2dos' or opening the
file in an editor that replaces LF by CRLF by default and blindly
answering 'yes' upon request to overwrite. Thus, the patch is not strictly
necessary as Steven points out in the PR, module files state: "If you edit
this, you'll get what you deserve" -- adding '\r' counts as changing the
file. However, the patch is non-intrusive, so there shouldn't be to much room
for regressions.