[Kazehakase-devel 2695] Re: gecko_version の取得方法変更のお願い

Back to archive index

Hidetaka Iwai tyuyu****@sabis*****
2006年 11月 11日 (土) 19:15:35 JST


岩井です。

At Sat, 11 Nov 2006 16:10:05 +0900,

> ここの code ですが、-Wconversion -Wsign-compare -Werror で暗黙の型変換
> を error にしておいて
> 
> AC_COMPILE_IFELSE(
> 	[AC_LANG_PROGRAM(
> 		[[#include <nsIHelperAppLauncherDialog.h>]],
> 		[[nsIHelperAppLauncherDialog *p;
> 		  p->Show (nsnull, nsnull, -1);]]
> 	)],
> 	[AC_DEFINE([MOZ_NSIHELPERAPPLAUNCHERDIALOG_NSPRBOOL_],[1],[Define if nsIHelperAppLauncharDialog uses PRBool]) result=yes],
> 	[result=no])
> 
> 
> で p->Show (nsnull, nsnull, -1) の第3引数で -1 が通らなければ第3引数は
> PRBool, 通れば PRUint32 という判定をしていると思うんですが、その後の
> action-if-true と action-if-false の内容が逆ではないですか?

書いたあとで気がつきましたが、-Wsign-compare だから -1 だと PRUint32 じゃ
駄目ですね。というか PRBool が char の typedef のなので、-1 でも
PRBool としては問題ないのでした。

とりあえず -Werror があると xulrunner でビルドできない問題は添付したパッチの
ような姑息療法でしのいで、0.4.2 の Debian package を upload しました。

regards,
-- 
 Hidetaka Iwai
 tyuyu****@sabis*****

-------------- next part --------------
テキスト形式以外の添付ファイルを保管しました...
ファイル名: 60_fix_ftbfs.dpatch
型:         application/octet-stream
サイズ:     2068 バイト
説明:       無し
Download 


Kazehakase-devel メーリングリストの案内
Back to archive index