Re: [PATCH v10 12/12] bisect--helper: `get_terms` & `bisect_terms` shell function in C
- From
Torsten Bögershausen <tboegi@web.de>
- Date
- Jul 26, 2016, 01:42 UTC
- Message-ID
- <f6657922-555e-affa-8904-678bc7ee3f71@web.de>
- In-Reply-To
- <xmqqh9bd7j6k.fsf@gitster.mtv.corp.google.com>
On 07/25/2016 06:53 PM, Junio C Hamano wrote:
Show 20 quoted lines
> Pranit Bauva <pranit.bauva@gmail.com> writes:
>
>>>> >>> +enum terms_defined {
>>>> >>> + TERM_BAD = 1,
>>>> >>> + TERM_GOOD = 2,
>>>> >>> + TERM_NEW = 4,
>>>> >>> + TERM_OLD = 8
>>>> >>> +};
>>>> >>> +
>>> >>
>>> >> What does TERM stand for ?
> The terms (words) used to denote the newer and the older parts of
> the history. Traditionally, as a regression-hunting tool (i.e. it
> used to work, where did I break it?), we called older parts of the
> history "good" and newer one "bad", but as people gained experience
> with the tool, it was found that the pair of words was error-prone
> to use for an opposite use case "I do not recall fixing it, but it
> seems to have been fixed magically, when did that happen?", and a
> more explicit "new" and "old" were introduced.
>Thanks for the explanation. Is there any risk that a more generic term like "TERM_BAD" may collide with some other definition some day ?
Would it make sense to call it GIT_BISECT_TERM_BAD, GBS_TERM_BAD, BIS_TERM_BAD or something more unique ?