From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 02 Nov 2018 03:11:56 GMT
Subject: Re: [PATCH v3] i18n: make GETTEXT_POISON a runtime option
Message-ID: <xmqqlg6cqikj.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <20181101193115.32681-1-avarab@gmail.com>

Ævar Arnfjörð Bjarmason  <avarab@gmail.com> writes:

> Change the GETTEXT_POISON compile-time + runtime GIT_GETTEXT_POISON
> test parameter to only be a GIT_TEST_GETTEXT_POISON=<non-empty?>
> runtime parameter, to be consistent with other parameters documented
> in "Running tests with special setups" in t/README.
> ...
>  #ifndef NO_GETTEXT
>  extern void git_setup_gettext(void);
>  extern int gettext_width(const char *s);
>  #else
>  static inline void git_setup_gettext(void)
>  {
> +	use_gettext_poison();; /* getenv() reentrancy paranoia */
>  }

Too many "case/esac" statement in shell scripting?