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

Re: [PATCH 4/4] utf8.c: speculatively assume utf-8 in strbuf_add_wrapped_text()

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Feb 20, 2010, 09:14 UTC
Message-ID
<alpine.DEB.1.00.1002201012300.20986@pacific.mpi-cbg.de>
In-Reply-To
<4B7F0EBC.4060209@lsrfire.ath.cx>
Hi,
On Fri, 19 Feb 2010, René Scharfe wrote:
> Missing: numbers for a non-utf-8 repo.

Since the wrapping code only makes sense for UTF-8, I consider it a pilot error to try to rewrap non-UTF-8 messages. So I think you are safe.

All of the patches look good to me, Dscho

Previous: René ScharfeNext: Junio C Hamano
Message 6 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.