This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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: [PATCH, fortran] Code clean up patch.

From: Steve Kargl <sgk at troutmask dot apl dot washington dot edu>
To: Janus Weil <janus at gcc dot gnu dot org>
Cc: fortran at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
Date: Fri, 12 Jun 2009 14:15:31 -0700
Subject: Re: [PATCH, fortran] Code clean up patch.
References: <20090612203830.GA93458@troutmask.apl.washington.edu> <854832d40906121406o6c9d341bm69102256a3ac77b2@mail.gmail.com>

On Fri, Jun 12, 2009 at 11:06:04PM +0200, Janus Weil wrote:
> 2009/6/12 Steve Kargl <sgk@troutmask.apl.washington.edu>:
> > The attached patch was regression tested on x86_64-*-freebsd.
> > There were no regressions.
> >
> > It simply moves a function from arith.c to decl.c, where
> > decl.c is the only file that uses this function. ?I also
> > made the function static. ?In addition, the patch wraps
> > one long line. ?OK for trunk?
> 
> Ok.
> 
> However, one comment about the long line: I guess it was actually me
> who committed this line. And to me it looks like it is exactly 80
> characters long, so I'd say there is no need to wrap it?!?

Thanks.

Committed as r148441 without the line wrap.

-- 
steve

References:
- [PATCH, fortran] Code clean up patch.
  - From: Steve Kargl
- Re: [PATCH, fortran] Code clean up patch.
  - From: Janus Weil

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