masover-threadsafe 0.0.3 → 0.1.0

data/lib/threadsafe.rb

@@ -3,18 +3,39 @@
 #   (Otherwise, we'd either have to require them here, or intercept every
 #    require, which we can't do until autoload is fixed.)
 
-{
+unsafe_classes = [
   # Default to false
-  Object => false
-}.each_pair do |klass, safe|
+  Object
+]
+
+safe_classes = [
+  # Immutables
+  TrueClass,
+  FalseClass,
+  NilClass,
+  Symbol,
+  Numeric
+]
+
+# TODO: Are frozen strings safe? If so, what's the sanest implementation?
+# Since #freeze and #frozen? seem to exist on Object, it can't be global.
+
+
+# Unrolled to avoid ugly evals or define_method overhead.
+# It's simple enough anyway.
+
+safe_classes.each do |klass|
+  klass.class_eval do
+    unless method_defined? :threadsafe?
+      def threadsafe?; true; end
+    end
+  end
+end
+
+unsafe_classes.each do |klass|
   klass.class_eval do
     unless method_defined? :threadsafe?
-      # Hack to avoid define_method overhead
-      if safe
-        def threadsafe?; true; end
-      else
-        def threadsafe?; false; end
-      end
+      def threadsafe?; false; end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: masover-threadsafe
 version: !ruby/object:Gem::Version 
-  version: 0.0.3
+  version: 0.1.0
 platform: ruby
 authors: 
 - David Masover