Subject: Re: [PATCH] test-lib: fix "$remove_trash" regression and match_pattern_list() bugs

From: Junio C Hamano <[email-masked]>
Date: Thu, 17 Jun 2021 09:29:18 +0900

Jeff King <[email-masked]> writes:

> My thought was that the subshell takes us back to the original state,
> regardless of what it was. As opposed to "set +f" which takes us back to
> a particular state. But it is unlikely that we'd have done a global "set
> -f" before calling this, so maybe that is being overly conservative.

Overly conservative, yes, but if it is not too much overhead, I
think it is a good practice anyway.