From beaabd23087a54364bc8fc8aa7a45dd9f425e19b Mon Sep 17 00:00:00 2001 From: Takashi Kokubun Date: Tue, 3 Sep 2019 21:08:07 +0900 Subject: [PATCH] Unify SUPPORT_JOKE and OPT_SUPPORT_JOKE for simplicity and consistency. Now SUPPORT_JOKE needs to be prefixed with OPT_ to make the config visible in `RubyVM::VmOptsH`, and the inconsistency was introduced. As it has never been available for override in configure (no #ifndef guard), it should be fine to rename the config. --- compile.c | 4 ++-- eval.c | 2 +- iseq.h | 2 +- vm_opts.h | 1 - 4 files changed, 4 insertions(+), 5 deletions(-) diff --git a/compile.c b/compile.c index ad9998250d..3315716cc5 100644 --- a/compile.c +++ b/compile.c @@ -732,7 +732,7 @@ rb_iseq_compile_node(rb_iseq_t *iseq, const NODE *node) ADD_INSN(ret, ISEQ_COMPILE_DATA(iseq)->last_line, leave); } -#if SUPPORT_JOKE +#if OPT_SUPPORT_JOKE if (ISEQ_COMPILE_DATA(iseq)->labels_table) { st_table *labels_table = ISEQ_COMPILE_DATA(iseq)->labels_table; ISEQ_COMPILE_DATA(iseq)->labels_table = 0; @@ -6570,7 +6570,7 @@ compile_call(rb_iseq_t *iseq, LINK_ANCHOR *const ret, const NODE *const node, in INIT_ANCHOR(recv); INIT_ANCHOR(args); -#if SUPPORT_JOKE +#if OPT_SUPPORT_JOKE if (nd_type(node) == NODE_VCALL) { ID id_bitblt; ID id_answer; diff --git a/eval.c b/eval.c index 614bb043ee..30d4ea57c3 100644 --- a/eval.c +++ b/eval.c @@ -478,7 +478,7 @@ static VALUE get_ec_errinfo(const rb_execution_context_t *ec); static VALUE exc_setup_cause(VALUE exc, VALUE cause) { -#if SUPPORT_JOKE +#if OPT_SUPPORT_JOKE if (NIL_P(cause)) { ID id_true_cause; CONST_ID(id_true_cause, "true_cause"); diff --git a/iseq.h b/iseq.h index 13cbd90569..711afeb0fa 100644 --- a/iseq.h +++ b/iseq.h @@ -109,7 +109,7 @@ struct iseq_compile_data { unsigned int ci_kw_index; const rb_compile_option_t *option; struct rb_id_table *ivar_cache_table; -#if SUPPORT_JOKE +#if OPT_SUPPORT_JOKE st_table *labels_table; #endif }; diff --git a/vm_opts.h b/vm_opts.h index 9892d3c999..d4e2ed786e 100644 --- a/vm_opts.h +++ b/vm_opts.h @@ -62,7 +62,6 @@ /* misc */ #define OPT_SUPPORT_JOKE 0 -#define SUPPORT_JOKE OPT_SUPPORT_JOKE #ifndef VM_COLLECT_USAGE_DETAILS #define VM_COLLECT_USAGE_DETAILS 0