[Ruby 1.9-Bug#4143][Open] warning: "SUPPORT_JOKE" is not defined

Bug #4143: warning: “SUPPORT_JOKE” is not defined
http://redmine.ruby-lang.org/issues/show/4143

起票者: Kazuhiro NISHIYAMA
ステータス: Open, 優先度: Normal
カテゴリ: core
ruby -v: ruby 1.9.3dev (2010-12-10 trunk 30153) [x86_64-linux]

chkbuild で以前から
id.h:110:5: warning: “SUPPORT_JOKE” is not defined
id.h:116:5: warning: “SUPPORT_JOKE” is not defined
という警告が出ているのですが、id.h で「#if SUPPORT_JOKE」として参照しているのに
SUPPORT_JOKE を #define している vm_opts.h を include せずに id.h だけを
include しているファイルがあるからのようです。
(debug.c のように vm_core.h 経由で vm_opts.h を include した後に id.h を
include しているものもある。)

というわけで以下の修正で警告は出なくなるようです。

diff --git a/template/id.h.tmpl b/template/id.h.tmpl
index b91292a…29548de 100644
— a/template/id.h.tmpl
+++ b/template/id.h.tmpl
@@ -50,6 +50,8 @@ end
#include “parse.h”
#endif

+#include “vm_opts.h” /* for SUPPORT_JOKE */
+
#define symIFUNC ID2SYM(idIFUNC)
#define symCFUNC ID2SYM(idCFUNC)

チケット #4143 が更新されました。 (by Nobuyoshi N.)

ステータス OpenからClosedに変更
進捗 % 0から100に変更

This issue was solved with changeset r30165.
Kazuhiro, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

チケット #4143 が更新されました。 (by Kazuhiro NISHIYAMA)

ステータス ClosedからOpenに変更

説明に書いたように警告が出るのはちゃんと vm_opts.h の定義が反映されていないからなので、無視するだけだと、 vm_opts.h で

#define SUPPORT_JOKE 1

にしたときに SUPPORT_JOKE が有効な状態でコンパイルされるファイルと無効な状態でコンパイルされるファイル (がほとんど)
があってまずいです。

2010$BG/(B12$B7n(B10$BF|(B22:38 Kazuhiro NISHIYAMA
[email protected]:

http://redmine.ruby-lang.org/issues/show/4143
$B!J$$=$C!K(BSUPPORT_JOKE feature$B$r:o=|$7$F$7$^$&(B

/me run away.

(2010/12/10 23:49), KOSAKI Motohiro wrote:

$B!J$$=$C!K(BSUPPORT_JOKE feature$B$r:o=|$7$F$7$^$&(B
+1

2010/12/10 SASADA Koichi [email protected]:

(2010/12/10 23:49), KOSAKI Motohiro wrote:

$B!J$$=$C!K(BSUPPORT_JOKE feature$B$r:o=|$7$F$7$^$&(B
+1

SUPPORT_JOKE feature
$B$O!“;d$K?<9o$K$J$C$F$O$$$1$J$$!”$H8@$$J9$+$;$F$$$k$h$&$K;W$($F$J$j$^$;$s!#(B

チケット #4143 が更新されました。 (by Anonymous)

ステータス OpenからClosedに変更

This issue was solved with changeset r30184.
Kazuhiro, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

(2010/12/11 3:19), Aaron P. wrote:

SUPPORT_JOKE feature
$B$O!";d$K?<9o$K$J$C$F$O$$$1$J$$!"$H8@$$J9$+$;$F$$$k$h$&$K;W$($F$J$j$^$;$s!#(B

$B!!$=$&$$$&$H$3$m$b$"$k$s$@$H;W$$$^$9$,!$:#$N$H$3$m!$$=$b$=$b<BAu$b%P%0$C(B
$B$F$$$k$H$$$&1=$b$"$k$N$G!$>C$7$A$c$&$N$,$$$$$N$+$J$!!$$H;W$$$^$9!%(B

require ‘joke’ $B$_$?$$$J$N$,$"$k$H$$$$$N$+$b$7$l$J$$!%(B