Re: [PATCH] pretty: Fix bug in truncation support for %>, %< and %>

2013-04-29 Thread Ramsay Jones
Duy Nguyen wrote: > On Sun, Apr 28, 2013 at 2:43 AM, Ramsay Jones > wrote: >> The truncation support was implemented with the assistance of a >> new strbuf function (strbuf_utf8_replace). This function contains >> the following code: >> >>strbuf_attach(sb_src, strbuf_detach(&sb_dst, NULL),

Re: [PATCH] pretty: Fix bug in truncation support for %>, %< and %>

2013-04-29 Thread Jonathan Nieder
Hi, Ramsay Jones wrote: > Some systems experience failures in t4205-*.sh (tests 18-20, 27) > which all relate to the use of truncation with the %< padding > placeholder. This capability was added in the commit a7f01c6b > ("pretty: support truncating in %>, %< and %><", 19-04-2013). This is repro

Re: [PATCH] pretty: Fix bug in truncation support for %>, %< and %>

2013-04-27 Thread Duy Nguyen
On Sun, Apr 28, 2013 at 2:43 AM, Ramsay Jones wrote: > The truncation support was implemented with the assistance of a > new strbuf function (strbuf_utf8_replace). This function contains > the following code: > >strbuf_attach(sb_src, strbuf_detach(&sb_dst, NULL), > sb_

[PATCH] pretty: Fix bug in truncation support for %>, %< and %>

2013-04-27 Thread Ramsay Jones
Some systems experience failures in t4205-*.sh (tests 18-20, 27) which all relate to the use of truncation with the %< padding placeholder. This capability was added in the commit a7f01c6b ("pretty: support truncating in %>, %< and %><", 19-04-2013). The truncation support was implemented with th