Changeset 214

Timestamp:

09/23/2007 17:52:25 (15 months ago)

Author:

zimbatm

Message:

-50 octets gained by removing Camping::Apps.

Camping::Apps was only used by camping/db.rb. The array would also get filled
on each app reload, thus preventing the garbage collection of the old version.

Location:

trunk/lib

Files:

• camping-unabridged.rb (modified) (2 diffs)
• camping.rb (modified) (2 diffs)
• camping/db.rb (modified) (1 diff)

trunk/lib/camping-unabridged.rb

@@ -91 +91 @@
   #   Camping::Apps # => [Blog, Tepee]
   # 
-  Apps = []
   C = self
   S = IO.read(__FILE__).sub(/^  S = I.+$/,'')
@@ -613 +612 @@
     #
     def goes(m)
-      eval S.gsub(/Camping/,m.to_s).gsub("A\pps = []","Cam\ping::Apps<<self"), TOPLEVEL_BINDING
+      eval S.gsub(/Camping/,m.to_s), TOPLEVEL_BINDING
     end
 

trunk/lib/camping.rb

@@ -1 +1 @@
 %w[active_support markaby tempfile uri].map{|l|require l}
-module Camping;Apps=[];C=self;S=IO.read(__FILE__).sub(/S=I.+$/,'')
+module Camping;C=self;S=IO.read(__FILE__).sub(/S=I.+$/,'')
 P="Cam\ping Problem!";module Helpers;def R(c,*g);p,h=/\(.+?\)/,g.grep(Hash)
 (g-=h).inject(c.urls.find{|x|x.scan(p).size==g.size}.dup){|s,a|s.sub p,C.
@@ -36 +36 @@
 class ServerError<R();def get k,m,e;r(500,Mab.new{h1 P;h2"#{k}.#{m}";h3"#{e.class
 } #{e.message}:";ul{e.backtrace.each{|bt|li(bt)}}}.to_s)end end;self;end;class<<
-self;def goes m;eval S.gsub(/Camping/,m.to_s).gsub("A\pps=[]","Cam\ping::Apps<<\
-self"),TOPLEVEL_BINDING;end;def escape s;s.to_s.gsub(/[^ \w.-]+/n){'%'+($&.
+self;def goes m;eval S.gsub(/Camping/,m.to_s),TOPLEVEL_BINDING;end;def escape s
+s.to_s.gsub(/[^ \w.-]+/n){'%'+($&.
 unpack('H2'*$&.size)*'%').upcase}.tr(' ','+')end;def un s;s.tr('+',' ').gsub(
 /%([\da-f]{2})/in){[$1].pack('H*')}end;def qsp q,d='&;',y=nil,z=H[];m=proc{|_,o,n|o.u(

trunk/lib/camping/db.rb

@@ -72 +72 @@
   end
 end
-Camping::S.sub! "autoload:Base,'camping/db'", ""
-Camping::S.sub! "def Y;self;end", $AR_EXTRAS
-Camping::Apps.each do |app|
-    app::Models.module_eval $AR_EXTRAS
+Camping::S.sub! /autoload\s*:Base\s*,\s*['"]camping\/db['"]/, ""
+Camping::S.sub! /def\s*Y[;\s]*self[;\s]*end/, $AR_EXTRAS
+Object.constants.map{|c|Object.const_get(c)}.each do |c|
+  c::Models.module_eval $AR_EXTRAS if c.respond_to?(:run)
 end