On Mon, May 2, 2016 at 7:49 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Shin Kojima <shin@xxxxxxxxxx> writes:
>
>> This patch prepare git blob objects to be encoded into UTF-8 before
>> highlighting in the manner of `to_utf8` subroutine.
>> ---
>
> The single liner Perl invoked from the script felt a bit too dense
> to my taste but other than that I have no complaints to what the
> patched code does.
>
> Jakub, does it look good to you, too?
Yes, it looks all right to me. $^X is current Perl. -CO means that
the output is utf8 (for `highlight` command), -p means read all lines
and print them (it could be replaced by "print" command in one-liner),
-s is here to pass $fallback_encoding as $fe (it could be replaced,
but it would require some fiddling with quoting $s), -e '...' means
execute one line.
> Please sign-off your patch (see Documentation/SubmittingPatches).
>
> Thanks.
>
>
>> gitweb/gitweb.perl | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
>> index 05d7910..2fddf75 100755
>> --- a/gitweb/gitweb.perl
>> +++ b/gitweb/gitweb.perl
>> @@ -3935,6 +3935,9 @@ sub run_highlighter {
>>
>> close $fd;
>> open $fd, quote_command(git_cmd(), "cat-file", "blob", $hash)." | ".
>> + quote_command($^X, '-CO', '-MEncode=decode,FB_DEFAULT', '-pse',
>> + '$_ = decode($fe, $_, FB_DEFAULT) if !utf8::decode($_);',
>> + '--', "-fe=$fallback_encoding")." | ".
>> quote_command($highlight_bin).
>> " --replace-tabs=8 --fragment --syntax $syntax |"
>> or die_error(500, "Couldn't open file or run syntax highlighter");
--
Jakub Narebski
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html