From: Junio C Hamano Date: Fri, 27 Oct 2017 18:19:37 GMT Subject: Re: [PATCH v16 Part II 2/8] bisect--helper: `bisect_write` shell function in C Message-ID: In-Reply-To: <0102015f5e5ee22c-ea9c3a38-5d42-4dce-a54b-45c59768a70b-000000@eu-west-1.amazonses.com> Pranit Bauva writes: > - bisect_write "$state" "$rev" > + git bisect--helper --bisect-write "$state" "$rev" "$TERM_GOOD" "$TERM_BAD" || exit I can see why two extra "terms" parameters need to be passed to this helper at this step; looking at patches around 4/8 and 6/8 where C code can directly find out what words are used for GOOD and BAD, we should be able to lose these two extra parameters from this helper by internally making a call to get_terms() from bisect_write() ;-)