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: [gfortran] Minor cleanup


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.

- Tobi


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