Bug #4143: warning: "SUPPORT_JOKE" is not defined

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: ruby 1.9.3dev (2010-12-10 trunk 30153) [x86_64-linux]

[znz (Kazuhiro NISHIYAMA)]

=begin
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)
=end

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

=begin
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.

=end

[znz (Kazuhiro NISHIYAMA)]

• Status changed from Closed to Open

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

#define SUPPORT_JOKE 1

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

[kosaki (Motohiro KOSAKI)]

=begin
2010年12月10日22:38 Kazuhiro NISHIYAMA [e-mail]:

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

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

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

#define SUPPORT_JOKE 1

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

http://redmine.ruby-lang.org/issues/show/4143

（ぼそっ）SUPPORT_JOKE featureを削除してしまう

/me run away.

=end

[ko1 (Koichi Sasada)]

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

（ぼそっ）SUPPORT_JOKE featureを削除してしまう
+1

--
// SASADA Koichi at atdot dot net

=end

• Status changed from Open to Closed

=begin
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.

=end

[ko1 (Koichi Sasada)]

=begin
(2010/12/11 3:19), Aaron Patterson wrote:

SUPPORT_JOKE feature は、私に深刻になってはいけない、と言い聞かせているように思えてなりません。

　そういうところもあるんだと思いますが，今のところ，そもそも実装もバグっ
ているという噂もあるので，消しちゃうのがいいのかなぁ，と思います．

require 'joke' みたいなのがあるといいのかもしれない．

--
// SASADA Koichi at atdot dot net

=end