Re: Fix UTF Encoding issue
- From
Benjamin Close <benjamin.close@clearchain.com>
- Date
- Dec 3, 2007, 23:04 UTC
- Message-ID
- <20071203230432.GA1337@wolf.clearchain.com>
- In-Reply-To
- <200712040020.26773.ismail@pardus.org.tr>
On Tue, Dec 04, 2007 at 12:20:26AM +0200, Ismail D??nmez wrote:
Show 33 quoted lines
> Monday 03 December 2007 Tarihinde 23:46:24 yazm????t??:
> > Jakub Narebski wrote:
> > > On Mon, 3 Dec 2007, Martin Koegler wrote:
> > >> On Mon, Dec 03, 2007 at 04:06:48AM -0800, Jakub Narebski wrote:
> > >>> Ismail D??nmez <ismail@pardus.org.tr> writes:
> > >>>> Monday 03 December 2007 Tarihinde 12:14:43 yazm??t?:
> > >>>>> Benjamin Close <Benjamin.Close@clearchain.com> writes:
> > >>>>>> - eval { $res = decode_utf8($str, Encode::FB_CROAK); };
> > >>>>>> - if (defined $res) {
> > >>>>>> - return $res;
> > >>>>>> - } else {
> > >>>>>> - return decode($fallback_encoding, $str, Encode::FB_DEFAULT);
> > >>>>>> - }
> > >>>>>> + eval { return ($res = decode_utf8($str, Encode::FB_CROAK)); };
> > >>>>>> + return decode($fallback_encoding, $str, Encode::FB_DEFAULT);
> > >>>>>> }
> > >>
> > >> This version is broken on Debian sarge and etch. Feeding a UTF-8 and a
> > >> latin1 encoding of the same character sequence yields to different
> > >> results.
> >
> > For the record, this was on a debian sid machine.
> >
> > #perl --version
> > This is perl, v5.8.8 built for x86_64-linux-gnu-thread-multi
> >
> > and the result of not using the original patch was:
> >
> > <h1>Software error:</h1>
> > <pre>Cannot decode string with wide characters at
> > /usr/lib/perl/5.8/Encode.pm line 166. </pre>
>
> Can you try the attached patch?I confirm that the patch corrects the problem.
Without it I get the Cannot decode string error. With it gitweb displays correctly.
Cheers, Benjamin