This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [gfortran] PR libfortran/21376

From: FX Coudert <fxcoudert at gmail dot com>
To: Tobias Schlüter<tobias dot schlueter at physik dot uni-muenchen dot de>
Cc: gfortran <fortran at gcc dot gnu dot org>, gcc-patches at gcc dot gnu dot org, Steve Kargl <sgk at troutmask dot apl dot washington dot edu>
Date: Mon, 23 May 2005 23:09:57 +0200
Subject: Re: [gfortran] PR libfortran/21376
References: <4290D077.1060106@gmail.com> <4291128F.1090500@physik.uni-muenchen.de> <42917846.5000302@gmail.com> <429221FE.9070607@physik.uni-muenchen.de>

*Blush* well, yes.  Reminds me to never have the final look at a patch late at
night :-(  I agree that the patch is the easiest way to do this then, you
should add a testcase for it, though.  The testcase I posted is not good
enough given that I got the exponents of the corner cases wrong.

OK. I will add a testcase.

Another issue is that your patch is only superficially different from the one
Peter Wainwright suggested when he filed the PR, so you should probably add
his name to the ChangeLog entry.

OK, so here is my question: is the name in the ChangeLog entry supposed to be the one of the commiter, the one of the author, or both?

FX

Follow-Ups:
- Re: [gfortran] PR libfortran/21376
  - From: Steve Kargl
- Re: [gfortran] PR libfortran/21376
  - From: Tobias Schlüter

References:
- [gfortran] PR libfortran/21376
  - From: FX Coudert
- Re: [gfortran] PR libfortran/21376
  - From: Tobias Schlüter
- Re: [gfortran] PR libfortran/21376
  - From: FX Coudert
- Re: [gfortran] PR libfortran/21376
  - From: Tobias Schlüter

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]