Re: Bug#38107: new patch (correct, this time) for man-db japanese support.

[Fumitoshi UKAI <ukai@debian.or.jp>]

Hi, 

At Wed, 15 Sep 1999 09:04:38 +0300,
Fabrizio Polacco <fab@prosa.it> wrote:

> > Hmm, As I checked your patch, it does not work correctly.  
> 
> oh, oh, wrong file attached (hummm)...
> sorry for the noise </shame>.

I'm afraid it also does not work correctly.

> +				int j;
> +				for ( j=0; lang_table[j].lang; j++ ) {
> +					if (( strncmp( lang_table[j].lang, lang
> +						, strlen( lang_table[j].lang)) == 0 )
> +					||  ( lang_table[j].lang[0] == '*' )) {
> +						roff_device = lang_table[j].device;
> +						troff = 1;

 I think this `troff = 1;' should be removed, because when troff = 1,
 a man page will not be showed using pager, but simply output to stdout.
 (see man.c:display() around 1790 lines)
 Without this `troff = 1', it seems works fine for me.
 Why do you `troff = 1' here?
 
> +						putenv( strappend ( 0
> +							,"LESSCHARSET="
> +							, lang_table[j].charset
> +							, 0));
> +						j = sizeof(lang_table)/sizeof(struct lt) - 2;
> +					}
> +				}
> +			}

Thanks
Fumitoshi UKAI