Merge pull request ruby#3493 from ruby/revert-3491-mark-as-ractor-safe

Revert "Mark extension as Ractor-safe"

Author: kddnewton

ext/prism/extension.c

@@ -1331,11 +1331,6 @@ Init_prism(void) {
         );
     }
 
-#ifdef HAVE_RB_EXT_RACTOR_SAFE
-    // Mark this extension as Ractor-safe.
-    rb_ext_ractor_safe(true);
-#endif
-
     // Grab up references to all of the constants that we're going to need to
     // reference throughout this extension.
     rb_cPrism = rb_define_module("Prism");

lib/prism/ffi.rb

@@ -7,10 +7,6 @@
 require "rbconfig"
 require "ffi"
 
-# We want to eagerly load this file if there are Ractors so that it does not get
-# autoloaded from within a non-main Ractor.
-require "prism/serialize" if defined?(Ractor)
-
 module Prism
   module LibRubyParser # :nodoc:
     extend FFI::Library
@@ -163,9 +159,6 @@ def self.with
     class PrismString # :nodoc:
       SIZEOF = LibRubyParser.pm_string_sizeof
 
-      PLATFORM_EXPECTS_UTF8 =
-        RbConfig::CONFIG["host_os"].match?(/bccwin|cygwin|djgpp|mingw|mswin|wince|darwin/i)
-
       attr_reader :pointer, :length
 
       def initialize(pointer, length, from_string)
@@ -200,7 +193,8 @@ def self.with_file(filepath)
         # On Windows and Mac, it's expected that filepaths will be encoded in
         # UTF-8. If they are not, we need to convert them to UTF-8 before
         # passing them into pm_string_mapped_init.
-        if PLATFORM_EXPECTS_UTF8 && (encoding = filepath.encoding) != Encoding::ASCII_8BIT && encoding != Encoding::UTF_8
+        if RbConfig::CONFIG["host_os"].match?(/bccwin|cygwin|djgpp|mingw|mswin|wince|darwin/i) &&
+           (encoding = filepath.encoding) != Encoding::ASCII_8BIT && encoding != Encoding::UTF_8
           filepath = filepath.encode(Encoding::UTF_8)
         end
 
@@ -229,7 +223,7 @@ def self.with_file(filepath)
   private_constant :LibRubyParser
 
   # The version constant is set by reading the result of calling pm_version.
-  VERSION = LibRubyParser.pm_version.read_string.freeze
+  VERSION = LibRubyParser.pm_version.read_string
 
   class << self
     # Mirror the Prism.dump API by using the serialization API.

templates/lib/prism/serialize.rb.erb

@@ -592,7 +592,7 @@ module Prism
       <%- tokens.each do |token| -%>
       <%= token.name.to_sym.inspect %>,
       <%- end -%>
-    ].freeze
+    ]
 
     private_constant :MAJOR_VERSION, :MINOR_VERSION, :PATCH_VERSION
     private_constant :ConstantPool, :FastStringIO, :Loader, :TOKEN_TYPES