[freewnn:00926] Re: defect in Japanese conversion

[aono@XXXX (Tomoki AONO)]

青野です。こんばんは。
＃小野さんが早く開発に復帰できるようにかげながら祈っております。

<86wuptg23s.wl@XXXX>の記事において
hiroo@XXXXさんは書きました。

>> 修正の前段階で、スタイル+コンパイラの warning に対する修正です。
>> (malloc の暫定対処も入っていますが、commit の際はそこは戻すつもり)。
>> local の cvs repo. に対する差分ですが、多分 cvs の最新のものに問題なく
>> あたるかと思います。
>> 
>> いつもは、make が通って起動出来たらつっこんでしまっているのですが、
>> -              fzk_orvt (&fzkwk[point + n], &(fzkinfp->vector[0]));
>> +              fzk_orvt (fzkwk[point + n].vector, &(fzkinfp->vector[0]));
>> といった struct FT, kangovect 関連の修正が一寸心配なので、review
>> お願いします。

ポインタのところは苦手なのであまりさわりたくないなあと思う
のですが、パッチを適用したjserverは今動作させていますので
何か分かれば適宜ご報告します。

頭の中だけで考えていてテストはしていませんが、例えば上記の例で

&(fzkinfp->vector[0]) ⇒ fzkinfp->vector

にするのはありでしょうか。(今のままでも問題はないと思いま
すが、ちょっとくどいかなと思った次第で…。)

それから、

>> diff -u -r1.1.1.1 fzk.c
>> --- fzk.c	13 Jun 2002 20:45:13 -0000	1.1.1.1
>> +++ fzk.c	10 Sep 2002 17:00:51 -0000

>> @@ -577,18 +588,17 @@
>>   * fzk_ckvt
>>   */
>>  int
>> -fzk_ckvt (vector)
>> -     register int vector;
>> +fzk_ckvt (int vector)
>>  {
>> -  return vector >= 0 && fzk_ck_vector (&ft->kango_vect_area[vector]);
>> +  return vector >= 0 &&
>> +	 fzk_ck_vector (&(ft->kango_vect_area)->vector[vector]);
>>  }

ここの

&(ft->kango_vect_area)->vector[vector]

が別のポインタを見ているような気がするので、変更前の意図す
るものを活かして、

ft->kango_vect_area[vector].vector

の方がいいのではないかと思います。
＃&(ft->...vector[0])でも構いません。
----
青野智樹	(aono@XXXX)
Personal opinion only..