[PATCH] getenv: Move call to strlen to the branch it's used in.

Florian Weimer fw@deneb.enyo.de
Sat Nov 2 15:20:00 GMT 2019


* Lode Willems:

> The len variable is only used in the else branch.
> We don't need the call to strlen if the name is 0 or 1 characters long.
>
> 2019-10-02  Lode Willems  <Lode.Willems@UGent.be>
>
> 	* tdlib/getenv.c: Move the call to strlen into the branch it's used.
>
> diff --git a/stdlib/getenv.c b/stdlib/getenv.c
> index 6be97b2a54..aa5e69d0b0 100644
> --- a/stdlib/getenv.c
> +++ b/stdlib/getenv.c
> @@ -32,7 +32,6 @@
>  char *
>  getenv (const char *name)
>  {
> -  size_t len = strlen (name);
>    char **ep;
>    uint16_t name_start;
>  
> @@ -63,6 +62,7 @@ getenv (const char *name)
>      }
>    else
>      {
> +      size_t len = strlen (name);
>  #if _STRING_ARCH_unaligned
>        name_start = *(const uint16_t *) name;
>  #else

Thanks for the patch.  Is there any particular reason for making the
change?  I don't think it will alter the generated machine code.



More information about the Libc-alpha mailing list