autoload 0.2.0 → 0.3.0

data/.index

@@ -38,11 +38,11 @@ copyrights:
   license: BSD-2-Clause
 summary: What Ruby does but how Ruby should do it.
 title: Autoload
-version: 0.2.0
+version: 0.3.0
 name: autoload
 description: ! 'Ruby is officially deprecating autoload. Well, some people may prefer
 
   to keep it! Moreover, Ruby''s implementation has had a bug in it
 
   for-like-ever. This gem fixes that problem too.'
-date: '2013-01-06'
+date: '2013-01-08'

data/HISTORY.md

@@ -1,5 +1,18 @@
 # RELEASE HISTORY
 
+## 0.3.0 / 2013-01-08
+
+Turns out calling `super` in `#const_missing` was a bug. Instead it needed to
+alias the old `#const_missing` and call that instead. Also, turned out the
+`Object.const_missing` definition wasn't needed. Seems Toplevel's proxy knows
+to look in Module for that.
+
+Changes:
+
+* Use alias instead of super call in Module#cont_missing.
+* Remove Object.const_missing.
+
+
 ## 0.2.0 / 2013-01-06
 
 Fixed broken namespace autoloads --damn thing seems rather tricker

data/demo/02_module.md

@@ -1,6 +1,6 @@
 ## Module#autoload
 
-An autoload can be used within a module for constants relative to its namespace. 
+Autoload can be used within a module for constants relative to its namespace. 
 
     module ::And
       autoload :Me, 'andme'
@@ -8,3 +8,8 @@ An autoload can be used within a module for constants relative to its namespace.
 
     ::And::Me  #=> true
 
+Note we have to the toplevel indicators here b/c `require 'andme'` will load
+the `And` module into the toplevel, but QED executes demos in a special 
+module's scope (which emulates the toplevel, but as can be seen from this
+demo, it is not possible to do so perfectly.)
+

data/lib/autoload.rb

@@ -1,71 +1,80 @@
+# Global table to track constants that need to be autoloaded if accessed.
 $AUTOLOAD = Hash.new{ |h, k| h[k] = [] }
 
 #
-# Toplevel autoload.
+# Toplevel autoload method.
+#
+# @param [#to_sym] cname 
+#   The constants name.
+#
+# @param [String] path
+#   File path to require.
+#
+# @return [String] The $AUTOLOAD table.
 #
 def self.autoload(cname, path)
   $AUTOLOAD[[Object, cname.to_sym]] << path
 end
 
-class Object
+module Kernel
   #
+  # Instance level autoload method.
   #
+  # Note: I am not so certain the instance level method is a good idea.
+  # The end used can just as easily and more cleary use `self.class.autoload`
+  # to do it themselves. Nonetheless, Ruby supported this so we will too.
   #
-  def self.const_missing(name)
-    if paths = $AUTOLOAD.delete([self, name])
-      paths.each do |path|
-        require(path)
-      end
-      super(name) unless const_defined?(name)
-      const_get(name)
-    else
-      super(name)
-    end
+  # @param [#to_sym] cname 
+  #   The constants name.
+  #
+  # @param [String] path
+  #   File path to require.
+  #
+  # @return [String] The $AUTOLOAD table.
+  #
+  def autoload(cname, path)
+    $AUTOLOAD[[self.class, cname.to_sym]] << path
   end
-
 end
 
 class Module
   #
-  # Module autoload.
+  # Module/Class level autoload method.
+  #
+  # @param [#to_sym] cname 
+  #   The constants name.
+  #
+  # @param [String] path
+  #   File path to require.
+  #
+  # @return [String] The $AUTOLOAD table.
   #
   def autoload(cname, path)
     $AUTOLOAD[[self, cname.to_sym]] << path
   end
 
+ private
+
+  alias :const_missing_without_autoload :const_missing
+
   #
+  # Check the $AUTOLOAD table for a `[self, name]` entry. If present,
+  # require file and try to get the constant again.
   #
+  # @param [#to_sym] cname 
+  #   The constants name.
   #
   def const_missing(name)
     if paths = $AUTOLOAD.delete([self, name])
       paths.each do |path|
         require(path)
       end
-      super(name) unless const_defined?(name)
+      const_missing_without_autoload(name) unless const_defined?(name)
       const_get(name)
     else
-      super(name)
+      const_missing_without_autoload(name)
     end
   end
-end
-
-class Class
-  #
-  # We define autoload on Class too, just to be sure.
-  #
-  def autoload(cname, path)
-    $AUTOLOAD[[self, cname.to_sym]] << path
-  end
-end
 
-module Kernel
-  #
-  # Note: I am not so certain the instance level method is a good idea.
-  # The end used can just as easily and more cleary use `self.class.autoload`
-  # to do it themselves.
-  #
-  def autoload(cname, path)
-    $AUTOLOAD[[self.class, cname.to_sym]] << path
-  end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: autoload
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-01-06 00:00:00.000000000 Z
+date: 2013-01-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: qed