nkf32dllがビルドできない

• Reporter: turutani
• Owner: (None)
• Type: Bugs
• Status: Closed
• Component: (None)
• MileStone: (None)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: Fixed
• File: None

Details Reply

WinXP上のVC++でビルドを試みると、

nmake -f nkf.mak
...(略)...
nkf32.c
nkf32dll.c(340) : warning C4090: 'function' : 'const'
修飾子は異なります。
nkf32dll.c(340) : warning C4024: 'options' : の型が 1
の仮引数および実引数と異な
ります。
nkf32dll.c(446) : error C2065: 'utf16_mode' : 定義され
ていない識別子です。
nkf32dll.c(446) : error C2065: 'UTF16BE_INPUT' : 定義さ
れていない識別子です。
NMAKE : fatal error U1077: 'cl' : ﾘﾀｰﾝ ｺｰﾄﾞ '0x2'
Stop.

となる。
今まではDLLのビルドが行われてきたのかも気にしていなかっ
たが、ともかくこのままではよくないらしい。
但し、nkf.exeはきちんとビルドされている。

とりあえず、446行目を消せば(そして前後を調整すれば)何と
かなるが、それでいいのか、よく分からない。

Ticket History (3/6 Histories)

2007-07-19 10:07 Updated by: naruse

Comment Reply

Logged In: YES
user_id=3487

あぁ、すいません。nkf32.dllはまったくチェックしていませんで
CVS Headは手元ではビルドできるようになっています。

2007-07-20 15:53 Updated by: turutani

Comment Reply

Logged In: YES
user_id=4385

うーん、なんとなくif...elseの関係がおかしい気がします。
これでもコンパイルできるのかもしれませんが。
それと、"//"のコメントも、一部のコンパイラで嫌われます。
Cのコードなので、なるべくきれいに書いてもらえると
助かるのですが...

2007-07-21 11:04 Updated by: naruse

Comment Reply

Logged In: YES
user_id=3487

if-elseは具体的にはどこです？ぱっと見ではみつけられませんで
した。
// は気づきませんでした、次のコミットでまとめて直します。
わたしが新たに書く部分は自分なりに綺麗に書いてるつもりなので
すが、昔のコードも多いのでなんとも。指摘していただくなり、パ
ッチなどくだされば反映はします。

2007-07-25 13:05 Updated by: turutani

Comment Reply

Logged In: YES
user_id=4385

失礼しました。
if-elseは私の勘違いだったようです。
また、nkf-2.0.8bでは、nkf.makやコメントの件も含めて修正され
ていることを確認しました。
ありがとうございました。

ところで、別件なのですが、nkf32dllに関することなので...
1. 最初に示したビルドログでの340行に関する警告は、
334行の関数の宣言で、LPCSTRではなくLPSTRを指定すれば
出なくなります。
constであることは間違いないのでしょうけど...
2. ビルド時に、WINやNTというディレクトリが無い、というエラー
が出ます。
特に理由がなければ、nkf.makから"copy"で始まる行を全て消して
もいいのではないかと思います。
もしもリリースの都合上で必要なら、最初から作成しておくか、ど
こかでmkdirするかにしたほうがいいと思います。

2007-07-25 22:15 Updated by: naruse

Comment Reply

Logged In: YES
user_id=3487

両者とも修正しました。
nkf.c の options が char* なので LPSTR が正しいですね。
不要な /LD も削除したので、
これであからさまなエラーは出なくなったかと思います。

2007-08-08 00:38 Updated by: naruse

• Ticket Close date is changed to 2007-08-08 00:38
• Resolution Update from None to Fixed
• Status Update from Open to Closed