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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH 4/4] utf8.c: speculatively assume utf-8 in strbuf_add_wrapped_text()
From: Johannes Schindelin <Johannes.Schindelin@xxxxxx>
Date: Sat, 20 Feb 2010 10:14:57 +0100 (CET)
Cc: Git Mailing List <git@xxxxxxxxxxxxxxx>
In-reply-to: <4B7F0EBC.4060209@xxxxxxxxxxxxxx>
User-agent: Alpine 1.00 (DEB 882 2007-12-20)

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

Follow-Ups:
- Re: [PATCH 4/4] utf8.c: speculatively assume utf-8 in strbuf_add_wrapped_text()
  - From: Junio C Hamano

References:
- [PATCH 0/4] utf8.c: strbuf'ify strbuf_add_wrapped_text()
  - From: René Scharfe
- [PATCH 4/4] utf8.c: speculatively assume utf-8 in strbuf_add_wrapped_text()
  - From: René Scharfe

Prev by Date: Re: [PATCH 0/4] utf8.c: strbuf'ify strbuf_add_wrapped_text()
Next by Date: [PATCH squash] Simplify test-terminal.perl
Previous by thread: [PATCH 4/4] utf8.c: speculatively assume utf-8 in strbuf_add_wrapped_text()
Next by thread: Re: [PATCH 4/4] utf8.c: speculatively assume utf-8 in strbuf_add_wrapped_text()
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel Development] [Gcc Help] [IETF Annouce] [DCCP] [Netdev] [Networking] [Security] [V4L] [Bugtraq] [Yosemite] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux SCSI] [Fedora Users]