This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [gfortran] Minor cleanup
- From: Paul Brook <paul at codesourcery dot com>
- To: Tobias Schlüter <tobias dot schlueter at physik dot uni-muenchen dot de>
- Cc: fortran at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Date: Tue, 31 Aug 2004 20:17:03 +0100
- Subject: Re: [gfortran] Minor cleanup
- Organization: CodeSourcery
- References: <200408311959.51919.paul@codesourcery.com> <4134CDFF.3010608@physik.uni-muenchen.de>
On Tuesday 31 August 2004 20:14, Tobias Schlüter wrote:
> Paul Brook wrote:
> > u = find_unit (ioparm.unit);
> > - if (u != NULL)
> > - return u;
> >
> > - return NULL;
> > + return u;
> > }
>
> Any reason not to go all the way, and say "return find_unit (...);" and
> remove the then unused variable u? I can do this if you agree that it's
> worthwhile.
No reason, I just didn't spot that. Go ahead.
Paul