From: Ramkumar Ramachandra <artagnon@gmail.com>
Date: Thu, 04 Aug 2011 09:11:26 GMT
Subject: Re: gitweb highligh generate illegal char when the content had utf-8 chars
Message-ID: <CALkWK0kURz_Rax-JOPT2sW1x4VrqWk=KDjUzzw4apacDo1UTyg@mail.gmail.com>
In-Reply-To: <20110804053123.GA13198@greatfirst.com>

Hi,

zzs writes:
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index dab89f2..48def38 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -6465,7 +6465,7 @@ sub git_blob {
>                        $nr++;
>                        $line = untabify($line);
>                        printf qq!<div class="pre"><a id="l%i" href="%s#l%i" class="linenr">%4i</a> %s</div>\n!,
> -                              $nr, esc_attr(href(-replay => 1)), $nr, $nr, $syntax ? $line : esc_html($line, -nbsp=>1);
> +                              $nr, esc_attr(href(-replay => 1)), $nr, $nr, $syntax ? to_utf8($line) : esc_html($line, -nbsp=>1);
>                }
>        }
>        close $fd

Looks sane, but I don't know enough about Gitweb to comment.
+CC: Jakub, who can probably say more.

Thanks.

-- Ram