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.


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?!?

Cheers,
Janus


> 2009-06-12 ?Steven G. Kargl ?<kargls@comcast.net>
>
> ? ? ? ?* arith.c (gfc_enum_initializer): Move function ...
> ? ? ? ?* decl.c: ... here. ?Remove gfc_ prefix and make static.
> ? ? ? ?(enumerator_decl): Update function call.
> ? ? ? ?* gfortran.h: Wrap long line. ?Remove gfc_enum_initializer
> ? ? ? ?prototype.
>
> --
> Steve
>


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