git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v4 10/13] bisect--helper: reimplement `bisect_state` & `bisect_head` shell functions in C

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 8, 2020, 21:38 UTC
Message-ID
<xmqq7dvd1yc2.fsf@gitster.c.googlers.com>
In-Reply-To
<20200701133504.18360-11-mirucam@gmail.com>
Miriam Rubio <mirucam@gmail.com> writes:
Show 12 quoted lines
> @@ -856,6 +858,81 @@ static int bisect_autostart(struct bisect_terms *terms)
>  
>  		return res;
>  }
> + int mark_for_removal(const char *refname, const struct object_id *oid,
> +			    int flag, void *cb_data)
> +{
> +	struct string_list *refs = cb_data;
> +	char *ref = xstrfmt("refs/bisect%s", refname);
> +	string_list_append(refs, ref);
> +	return 0;
> +}

Funny indentation at the beginning, and worse yet, triggers an error with -Werror=missing-prototypes.

What is the relationship between this one and the similarly-named on in bisect.c?

Previous: Miriam RubioNext: Miriam Rubio
Message 14 of 17 in “Finish converting git bisect to C part 2”
  1. 00/13 Finish converting git bisect to C part 2Miriam Rubio, Jul 1, 2020
  2. 01/13 bisect--helper: BUG() in cmd_*() on invalid subcommandMiriam Rubio, Jul 1, 2020
  3. 03/13 bisect--helper: introduce new `write_in_file()` functionMiriam Rubio, Jul 1, 2020
  4. Chris TorekJul 1, 2020
  5. Junio C HamanoJul 8, 2020
  6. 02/13 bisect--helper: use '-res' in 'cmd_bisect__helper' returnMiriam Rubio, Jul 1, 2020
  7. 05/13 bisect: call 'clear_commit_marks_all()' in 'bisect_next_all()'Miriam Rubio, Jul 1, 2020
  8. 04/13 bisect--helper: reimplement `bisect_autostart` shell function in CMiriam Rubio, Jul 1, 2020
  9. 06/13 bisect--helper: reimplement `bisect_next` and `bisect_auto_next` shell functions in CMiriam Rubio, Jul 1, 2020
  10. 08/13 bisect--helper: retire `--bisect-clean-state` subcommandMiriam Rubio, Jul 1, 2020
  11. 09/13 bisect--helper: retire `--next-all` subcommandMiriam Rubio, Jul 1, 2020
  12. 07/13 bisect--helper: finish porting `bisect_start()` to CMiriam Rubio, Jul 1, 2020
  13. 10/13 bisect--helper: reimplement `bisect_state` & `bisect_head` shell functions in CMiriam Rubio, Jul 1, 2020
  14. Junio C HamanoJul 8, 2020
  15. 11/13 bisect--helper: retire `--check-expected-revs` subcommandMiriam Rubio, Jul 1, 2020
  16. 12/13 bisect--helper: retire `--write-terms` subcommandMiriam Rubio, Jul 1, 2020
  17. 13/13 bisect--helper: retire `--bisect-autostart` subcommandMiriam Rubio, Jul 1, 2020

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.