From Bugzilla Helper: User-Agent: Mozilla/5.0 Galeon/1.2.0 (X11; Linux i686; U;) Gecko/20020408 Description of problem: less is broken for Japanese. The attached file should be seen as the attached png in Japanese mode. (LANG=ja_JP.eucJP, JLESSCHARSET=japanese) Version-Release number of selected component (if applicable): How reproducible: Always Steps to Reproduce: 1. less the attached file. 2. Oops. 3. Do you say we need to use emacs instead of less? RHL7.2 was better.. Actual Results: Broken - it shows hex dump. Expected Results: Human readable. Additional info:
Created attachment 57964 [details] 'ABC' file in Japanese multibyte.
Created attachment 57965 [details] It should be like this on kterm
I understand this iso patch breaks -N option for Japanese and other languages. And I found a mistake in the patch. The patch author added a line but forget to add {} to the for sentence. Below change fixed the problem. --- less-358/line.c.orig Mon May 20 18:02:51 2002 +++ less-358/line.c Mon May 20 18:03:06 2002 @@ -152,9 +152,10 @@ { sprintf(&linebuf[curr], "%*d", LINENUM_WIDTH, lno); column += LINENUM_WIDTH; - for (i = 0; i < LINENUM_WIDTH; i++) + for (i = 0; i < LINENUM_WIDTH; i++) { charset[curr] = ASCII; attr[curr++] = AT_NORMAL; + } } /* * Append enough spaces to bring us to the lmargin.
Created attachment 57984 [details] less-358+iso247-20001210-rh.diff (Updated iso patch for less)
Updated iso patch attached.
fixed in less-358-27 with iso254 and other patches