1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[DOC] Fix the comment for RUBY_CONST_ID and rb_intern

Browse Source 
RUBY_CONST_ID has never been deprecated; `rb_intern` is handy but it
is using non-standard GCC extensions and does not cache the ID with
other compilers.
This commit is contained in:
Nobuyoshi Nakada 2025-02-28 12:55:46 +09:00
parent 3294d3d457
commit 42c0722f83
No known key found for this signature in database
GPG Key ID: 3582D74E1FEE4465
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
include/ruby/internal/symbol.h
View File

@ -316,8 +316,9 @@ rbimpl_intern_const(ID *ptr, const char *str)
} }
/** /**
* Old implementation detail of rb_intern(). * Returns the cached ID for the given str in var, in compiler
* @deprecated Does anyone use it? Preserved for backward compat. * independent manner. Use this instead of GCC specific rb_intern()
* when you want to cache the ID on all platforms certainly.
*/ */
#define RUBY_CONST_ID(var, str) \ #define RUBY_CONST_ID(var, str) \
do { \ do { \
@ -326,7 +327,8 @@ rbimpl_intern_const(ID *ptr, const char *str)
} while (0) } while (0)
#if defined(HAVE_STMT_AND_DECL_IN_EXPR) #if defined(HAVE_STMT_AND_DECL_IN_EXPR)
/* __builtin_constant_p and statement expression is available /* GCC specific shorthand for RUBY_CONST_ID().
* __builtin_constant_p and statement expression is available
* since gcc-2.7.2.3 at least. */ * since gcc-2.7.2.3 at least. */
#define rb_intern(str) \ #define rb_intern(str) \
(RBIMPL_CONSTANT_P(str) ? \ (RBIMPL_CONSTANT_P(str) ? \