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

Re: [PATCH v3 03/12] bisect--helper: introduce new `write_in_file()` function

From
Đoàn Trần Công Danh <congdanhqx@gmail.com>
Date
May 23, 2020, 01:53 UTC
Message-ID
<20200523015331.GD3283@danh.dev>
In-Reply-To
<20200423070704.39872-4-mirucam@gmail.com>
On 2020-04-23 09:06:55+0200, Miriam Rubio <mirucam@gmail.com> wrote:
Show 9 quoted lines
> diff --git a/builtin/bisect--helper.c b/builtin/bisect--helper.c
> index 0fbd924aac..d3b2b33df0 100644
> --- a/builtin/bisect--helper.c
> +++ b/builtin/bisect--helper.c
> @@ -74,6 +74,28 @@ static int one_of(const char *term, ...)
>  	return res;
>  }
>  
> +static int write_in_file(const char *path, const char *mode, const char *format,...)
Nit: It looks like we want to add a space in the end ", ...)"
Since "one_of" above also has that space.
Otherwise, look good.
-- 
Danh
Previous: Johannes SchindelinNext: Miriam Rubio
Message 6 of 28 in “Finish converting git bisect to C part 2”
  1. 00/12 Finish converting git bisect to C part 2Miriam Rubio, Apr 23, 2020
  2. 01/12 bisect--helper: fix `cmd_*()` function switch default returnMiriam Rubio, Apr 23, 2020
  3. Johannes SchindelinMay 22, 2020
  4. 03/12 bisect--helper: introduce new `write_in_file()` functionMiriam Rubio, Apr 23, 2020
  5. Johannes SchindelinMay 22, 2020
  6. Đoàn Trần Công DanhMay 23, 2020
  7. 02/12 bisect--helper: use '-res' in 'cmd_bisect__helper' returnMiriam Rubio, Apr 23, 2020
  8. Johannes SchindelinMay 22, 2020
  9. 04/12 bisect--helper: reimplement `bisect_autostart` shell function in CMiriam Rubio, Apr 23, 2020
  10. Johannes SchindelinMay 22, 2020
  11. Johannes SchindelinMay 22, 2020
  12. 05/12 bisect--helper: reimplement `bisect_next` and `bisect_auto_next` shell functions in CMiriam Rubio, Apr 23, 2020
  13. Johannes SchindelinMay 22, 2020
  14. 07/12 bisect--helper: retire `--bisect-clean-state` subcommandMiriam Rubio, Apr 23, 2020
  15. 06/12 bisect--helper: finish porting `bisect_start()` to CMiriam Rubio, Apr 23, 2020
  16. Johannes SchindelinMay 22, 2020
  17. 08/12 bisect--helper: retire `--next-all` subcommandMiriam Rubio, Apr 23, 2020
  18. 10/12 bisect--helper: retire `--check-expected-revs` subcommandMiriam Rubio, Apr 23, 2020
  19. 09/12 bisect--helper: reimplement `bisect_state` & `bisect_head` shell functions in CMiriam Rubio, Apr 23, 2020
  20. Johannes SchindelinMay 22, 2020
  21. Miriam R.Jun 20, 2020
  22. Johannes SchindelinJun 19, 2020
  23. 11/12 bisect--helper: retire `--write-terms` subcommandMiriam Rubio, Apr 23, 2020
  24. 12/12 bisect--helper: retire `--bisect-autostart` subcommandMiriam Rubio, Apr 23, 2020
  25. Junio C HamanoApr 23, 2020
  26. Miriam R.Apr 25, 2020
  27. Johannes SchindelinMay 22, 2020
  28. Miriam R.May 24, 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.