diff options
| author | 卜部昌平 <shyouhei@ruby-lang.org> | 2020-07-22 15:37:28 +0900 |
|---|---|---|
| committer | 卜部昌平 <shyouhei@ruby-lang.org> | 2020-08-27 16:42:06 +0900 |
| commit | 5ca44076f47dce3a70b5f4755f5195f0f9093fc4 (patch) | |
| tree | d812ed884ac93ee97b53ae114bc456f07a008ebb | |
| parent | 2038cc6cab6ceeffef3ec3a765c70ae684f829ed (diff) | |
| download | ruby-5ca44076f47dce3a70b5f4755f5195f0f9093fc4.tar.gz | |
include/ruby/backward/2/rmodule.h: deprecate
Only one function in only one file uses contents of this public header.
That is not a wise idea. Let's just free the header's soul.
| -rw-r--r-- | class.c | 6 | ||||
| -rw-r--r-- | include/ruby/backward/2/rmodule.h | 6 | ||||
| -rw-r--r-- | include/ruby/ruby.h | 1 |
3 files changed, 9 insertions, 4 deletions
@@ -1027,7 +1027,7 @@ include_modules_at(const VALUE klass, VALUE c, VALUE module, int search_super) VALUE super_class = RCLASS_SUPER(c); // invalidate inline method cache - tbl = RMODULE_M_TBL(module); + tbl = RCLASS_M_TBL(module); if (tbl && rb_id_table_size(tbl)) { if (search_super) { // include if (super_class && !RB_TYPE_P(super_class, T_MODULE)) { @@ -1070,11 +1070,11 @@ include_modules_at(const VALUE klass, VALUE c, VALUE module, int search_super) VALUE refined_class = rb_refinement_module_get_refined_class(klass); - rb_id_table_foreach(RMODULE_M_TBL(module), add_refined_method_entry_i, (void *)refined_class); + rb_id_table_foreach(RCLASS_M_TBL(module), add_refined_method_entry_i, (void *)refined_class); FL_SET(c, RMODULE_INCLUDED_INTO_REFINEMENT); } - tbl = RMODULE_CONST_TBL(module); + tbl = RCLASS_CONST_TBL(module); if (tbl && rb_id_table_size(tbl)) constant_changed = 1; skip: module = RCLASS_SUPER(module); diff --git a/include/ruby/backward/2/rmodule.h b/include/ruby/backward/2/rmodule.h index 81ce91395f..a3e2d39f35 100644 --- a/include/ruby/backward/2/rmodule.h +++ b/include/ruby/backward/2/rmodule.h @@ -28,4 +28,10 @@ #define RMODULE_CONST_TBL(m) RCLASS_CONST_TBL(m) #define RMODULE_M_TBL(m) RCLASS_M_TBL(m) #define RMODULE_SUPER(m) RCLASS_SUPER(m) + +#if defined(__GNUC__) +# warning RMODULE_* macros are deprecated +#elif defined(_MSC_VER) +# pragma message("warning: RMODULE_* macros are deprecated") +#endif #endif /* RUBY_BACKWARD2_RMODULE_H */ diff --git a/include/ruby/ruby.h b/include/ruby/ruby.h index 33d7528118..c68168d500 100644 --- a/include/ruby/ruby.h +++ b/include/ruby/ruby.h @@ -51,7 +51,6 @@ #include "ruby/backward/2/assume.h" #include "ruby/backward/2/inttypes.h" #include "ruby/backward/2/limits.h" -#include "ruby/backward/2/rmodule.h" RBIMPL_SYMBOL_EXPORT_BEGIN() |