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

[PATCH 0/4] utf8.c: strbuf'ify strbuf_add_wrapped_text()

From
René Scharfe <rene.scharfe@lsrfire.ath.cx>
Date
Feb 19, 2010, 22:13 UTC
Message-ID
<4B7F0D08.6040608@lsrfire.ath.cx>

The strbuf parameter of strbuf_add_wrapped_text() in utf8.c is optional; if it's missing, results are directly written to stdout. By removing this flexibility and converting it to only call "pure" strbuf functions, this series cleans up the code a bit and enables an optimization, which is added in the last patch:

  [PATCH 1/4] utf8.c: remove print_wrapped_text()
  [PATCH 2/4] utf8.c: remove print_spaces()
  [PATCH 3/4] utf8.c: remove strbuf_write()
  [PATCH 4/4] utf8.c: speculatively assume utf-8 in strbuf_add_wrapped_text()
 builtin-shortlog.c |   17 ++++++++++++--
 utf8.c             |   61 +++++++++++++++++++++++----------------------------
 utf8.h             |    1 -
 3 files changed, 42 insertions(+), 37 deletions(-)
Next: René Scharfe
Message 1 of 8 in “utf8.c: strbuf'ify strbuf_add_wrapped_text()”
  1. 0/4 utf8.c: strbuf'ify strbuf_add_wrapped_text()René Scharfe, Feb 19, 2010
  2. 1/4 utf8.c: remove print_wrapped_text()René Scharfe, Feb 19, 2010
  3. 2/4 utf8.c: remove print_spaces()René Scharfe, Feb 19, 2010
  4. 3/4 utf8.c: remove strbuf_write()René Scharfe, Feb 19, 2010
  5. 4/4 utf8.c: speculatively assume utf-8 in strbuf_add_wrapped_text()René Scharfe, Feb 19, 2010
  6. Johannes SchindelinFeb 20, 2010
  7. Junio C HamanoFeb 20, 2010
  8. Johannes SchindelinFeb 20, 2010

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.