Ensure all code paths for interning symbols in C preserve the symbol.

aardvark179 · aardvark179 · commit fe7c51c12bc2 · 2022-09-15T14:43:31.000+01:00
diff --git a/src/main/c/cext/symbol.c b/src/main/c/cext/symbol.c
@@ -24,11 +24,11 @@ ID rb_intern(const char *string) {
 }
 
 ID rb_intern2(const char *string, long length) {
-  return SYM2ID(RUBY_INVOKE(rb_tr_temporary_native_string(string, length, rb_ascii8bit_encoding()), "intern"));
+  return SYM2ID(RUBY_CEXT_INVOKE("rb_intern", rb_tr_temporary_native_string(string, length, rb_ascii8bit_encoding())));
 }
 
 ID rb_intern3(const char *name, long len, rb_encoding *enc) {
-  return SYM2ID(RUBY_INVOKE(rb_tr_temporary_native_string(name, len, enc), "intern"));
+  return SYM2ID(RUBY_CEXT_INVOKE("rb_intern", rb_tr_temporary_native_string(name, len, enc)));
 }
 
 VALUE rb_sym2str(VALUE string) {

Original file line number	Diff line number	Diff line change
`@@ -24,11 +24,11 @@ ID rb_intern(const char *string) {`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`ID rb_intern2(const char *string, long length) {`
`27`		`- return SYM2ID(RUBY_INVOKE(rb_tr_temporary_native_string(string, length, rb_ascii8bit_encoding()), "intern"));`
	`27`	`+ return SYM2ID(RUBY_CEXT_INVOKE("rb_intern", rb_tr_temporary_native_string(string, length, rb_ascii8bit_encoding())));`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`ID rb_intern3(const char name, long len, rb_encoding enc) {`
`31`		`- return SYM2ID(RUBY_INVOKE(rb_tr_temporary_native_string(name, len, enc), "intern"));`
	`31`	`+ return SYM2ID(RUBY_CEXT_INVOKE("rb_intern", rb_tr_temporary_native_string(name, len, enc)));`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`VALUE rb_sym2str(VALUE string) {`