[Message Prev][Message Next][Thread Prev][Thread Next][Message Index][Thread Index]

Re: [MD:7127] 古いコンパイラでも Meadow を make する会



藤井です。

From: Kyotaro HORIGUCHI <horiguti@xxxxxxxxxxx>
Subject: Re: [MD:7127] 古いコンパイラでも Meadow を make する会
Date: Thu, 05 Jan 2006 19:31:32 +0900 (JST)
> > > > winuser.h の定義を mw32term.h にコピペするのではなく、winable.h をイン
> > > > クルードするのが良いのではないでしょうか。
> 
>  WinUser を include するとすごいことになるので持ってきてしまったの
> ですがこんなのがあるんですね^^; よかったよかった.

でも、将来的には winuser.h をインクルードすることになると思いますので、
その時には「すごいこと」をどうにか解決する必要があるかもしれません。

> > winable.h をじーーと見て個人的には納得していたところでした。
> 
> これを include して定義を取っ払えばいいんでしょう(適当)

これはその通りなんですが、SendInput API を呼び出すのが mw32term.c だけ
なのであれば、include と SendInputProc の定義は mw32term.c に移した方が
いいと思います。

あと、これまでの話と無関係ですが、rev4005 を見て一点気がついたことがあ
ります。

Vmw32_fep_switch_by_key_event という変数がありますが、この変数の定義に
は以下の問題があると思います。

  - 型は Lisp_Object ではなく、int でなければならない。([MD:6589] 参照)
  - 変数名の頭に "V" は不要。

あと、ブーリアンであるとはいえ、BOOL 型でもないのに FALSE を代入するの
は不自然な気がします。

ちなみに変数名の頭に不要な "V" が付いているのは他にもあるようです。

$ grep -n "DEFVAR_BOOL.*&V" *.c
callproc.c:1737:  DEFVAR_BOOL ("mw32-process-expects-pty", &Vmw32_process_expect
s_pty,
mw32ime.c:1864:  DEFVAR_BOOL ("mw32-fep-switch-by-key-event", &Vmw32_fep_switch_
by_key_event,

両方とも堀口さんが導入されたもののようですので、変数定義の見直しをお願い
します。

--
藤井 正行 / Masayuki FUJII