bleak_house 3.0.1 → 3.1

data/CHANGELOG

@@ -1,29 +1,8 @@
 
-v3.0.1. Rewrite. Remove Gruffs and graphs. Become a standard compiled extension. Walk the symbol table. Track object histories. Use custom CSV format instead of YAML.
+BleakHouse Changelog
 
-v2.7. Version renumbering; apologies if this messes anyone up.
-
-v2.6. RDoc documentation; use new Echoe setup; fix for namespaced controllers.
-
-v2.5. Detect if core keys are present or not; fixes for analyzing non-Rails apps.
-
-v2.4. Log heap usage; remove pure-Ruby profiler.
-
-v2.3. Subtract Rails core counts from action counts and join paired frames.
-
-v2.2. Change smoothing algorithm to be more intuitive and so mem usage is correct.
-
-v2.1. Close file properly.
-
-v2. C instrumentation with YAML output.
-
-v1.3. Log memory usage on *nix.
-
-v1.2. Fix nil directory traversal error on Windows.
-
-v1.1. Gem version
-
-v1. Dumped Rublique; replaced with a lightweight non-delta marshalling version.
-
-v0. First version
+3.1. fix nil directory traversal error on windows
+3. gem version
+2. dumped rublique once I understood what it did; replaced with a lightweight non-delta marshalling version
+1. first version
 

data/Manifest

@@ -1,30 +1,18 @@
-TODO
-test/unit/test_bleak_house.rb
-test/misc/direct.rb
-README
-Rakefile
-patches/parse.y.patch
-patches/gc.c.patch
-Manifest
-LICENSE_RUBY
-LICENSE_BSD
-LICENSE
-lib/vendor/lightcsv.rb
-lib/bleak_house.rb
-lib/bleak_house/support/rake.rb
-lib/bleak_house/support/core_extensions.rb
-lib/bleak_house/rails.rb
-lib/bleak_house/rails/dispatcher.rb
-lib/bleak_house/rails/bleak_house.rb
-lib/bleak_house/rails/action_controller.rb
-lib/bleak_house/logger.rb
-lib/bleak_house/logger/mem_usage.rb
-lib/bleak_house/analyzer.rb
-lib/bleak_house/analyzer/analyzer.rb
-install.rb
-init.rb
-ext/bleak_house/logger/snapshot.h
-ext/bleak_house/logger/snapshot.c
-ext/bleak_house/logger/extconf.rb
-CHANGELOG
-bin/bleak
+./CHANGELOG
+./LICENSE
+./Manifest
+./README
+./Rakefile
+./init.rb
+./install.rb
+./lib/bleak_house
+./lib/bleak_house/action_controller.rb
+./lib/bleak_house/analyze.rb
+./lib/bleak_house/bleak_house.rb
+./lib/bleak_house/dispatcher.rb
+./lib/bleak_house/gruff_hacks.rb
+./lib/bleak_house/mem_logger.rb
+./lib/bleak_house/rake_task_redefine_task.rb
+./lib/bleak_house/support_methods.rb
+./lib/bleak_house.rb
+./tasks/bleak_house_tasks.rake

data/README

@@ -1,73 +1,20 @@
+-----------------------------------------------------------------
 
 BleakHouse
+You need the 'gruff' gem, and 'rmagick', and all that.
 
-A library for finding memory leaks. 
+To run:
+  RAILS_ENV=production BLEAK_HOUSE=true INTERVAL=5 ./script/server 
+Browse around in your app. Cause requests. Do stuff. Then:
+  RAILS_ENV=production rake bleak_house:analyze
+And then look in log/bleak_house/.
 
-== License
-
-Copyright 2007 Cloudburst, LLC. See the included LICENSE file. Portions copyright 2006 Eric Hodel and used with permission. See the included LICENSE_BSD file. Portions copyright 2007 TOMITA Masahiro and used with permission. See the included LICENSE_RUBY file.
-
-The public certificate for this gem is at http://rubyforge.org/frs/download.php/25331/evan_weaver-original-public_cert.pem.
-
-== Features
-
-* leak-proof C instrumentation
-* fast logging
-* complete tracking of object and symbol history
-* easy Rails integration
-
-== Requirements
-
-* A unix-like operating system
-* Ruby 1.8.6
-
-= Usage
-
-== Installation
-
-Install the gem:
-  sudo gem install bleak_house -v 3.0.1
-
-You need to compile the BleakHouse patched Ruby build. In the plugin directory, run:
-  rake ruby:build
-  
-This gives you a Ruby binary named <tt>ruby-bleak-house</tt> alongside your regular binary.
-
-== Profiling a Rails app
-
-To setup a Rails app for profiling, just <tt>require 'bleak_house'</tt> in <tt>config/environment.rb</tt>.
-
-Then, to engage the logger (ideally, in a live deployment situation), start a server instance as so:
-  RAILS_ENV=production BLEAK_HOUSE=true ruby-bleak-house ./script/server 
-
-Browse around manually, thrash your entire app with a script, target troublesome controllers/actions, etc. The BleakHouse logger will dump a huge amount of data to <tt>log/bleak_house_production.dump</tt>--keep an eye on your disk space.
-
-Then, to analyze the dump:
-  bleak path/to/log/bleak_house_production.dump
+Copyright (c) 2007 Cloudburst, LLC. See included LICENSE file. Portions
+copyright Tony Arcieri and Scott Barron and used under license.
   
-Be patient.
-
-= Extras
-
-== Using BleakHouse outside of Rails
-
-Just instantiate a BleakHouse::Logger object, and make calls to <tt>snapshot</tt> at appropriate times.
-
-== Troubleshooting
-
-If you see the error <tt>Symbol not found: _rb_gc_heaps_used</tt>, it means you installed the patched binary, but tried to profile the server with the regular Ruby binary.
-
-You may get library require errors if you install <tt>ruby-bleak-house</tt> 1.8.6 alongside a different verson of Ruby. You could try to patch your local version of Ruby instead, or you could just upgrade to 1.8.6, which has a good trackrecord of stability.
-
-== Reporting problems
-
-* http://rubyforge.org/forum/forum.php?forum_id=13983
-
-Patches and contributions are very welcome. Please note that contributors are required to assign copyright for their additions to Cloudburst, LLC.
-
-== Further resources
+I was such a shy little thing that I seldom dared to open my lips, and never 
+dared to open my heart, to anybody else.
+- Dickens, Bleak House
 
-* http://blog.evanweaver.com/articles/2007/05/12/let-me-hit-you-with-some-knowledge
-* http://blog.evanweaver.com/articles/2007/05/06/leak-proof-direct-heap-instrumentation-for-bleak_house
-* http://blog.evanweaver.com/articles/2007/04/28/bleak_house
+-----------------------------------------------------------------
 

data/Rakefile

@@ -1,73 +1,66 @@
+require 'rubygems'
+require 'rake'
+require 'lib/bleak_house/rake_task_redefine_task'
 
-require 'lib/bleak_house/support/rake'
+NAME = "bleak_house"
 
 begin
+  require 'rake/clean'
+  gem 'echoe', '>= 1.1'
   require 'echoe'
+  require 'fileutils'
+
+  AUTHOR = "Evan Weaver"
+  EMAIL = "evan at cloudbur dot st"
+  DESCRIPTION = "BleakHouse is a Rails plugin for finding memory leaks. It tracks ObjectSpace for your entire app, and produces charts of references by controller, by action, and by object class."
+  CHANGES = `cat CHANGELOG`[/^([\d\.]+\. .*)/, 1]
+  RUBYFORGE_NAME = "fauna"
+  GEM_NAME = "bleak_house" 
+  HOMEPATH = "http://blog.evanweaver.com"
+  RELEASE_TYPES = ["gem"]
+  REV = nil
+  VERS = `cat CHANGELOG`[/^([\d\.]+)\. /, 1]
+  CLEAN.include ['**/.*.sw?', '*.gem', '.config']
+  RDOC_OPTS = ['--quiet', '--title', "bleak_house documentation", "--opname", "index.html", "--line-numbers", "--main", "README", "--inline-source"]
+  
+  include FileUtils
+  
+  taskmsg = File.open(File.dirname(__FILE__) + "/tasks/bleak_house_tasks.rake").readlines
+  taskmsg = taskmsg[0..3] + [taskmsg[7][2..-1]] + taskmsg[9..-1]
   
-  Echoe.new("bleak_house") do |p|
-    p.author = "Evan Weaver" 
-    p.project = "fauna"
-    p.summary = "A library for finding memory leaks."
-    p.url = "http://blog.evanweaver.com/files/doc/fauna/bleak_house/"
-    p.docs_host = 'blog.evanweaver.com:~/www/bax/public/files/doc/'
-    p.rdoc_pattern = /^ext.*\.c|lib.*logger.*rb|analyzer|rails\/bleak_house|^README|^CHANGELOG|^TODO|^LICENSE|^COPYING$/
-    p.dependencies = ['rake']
-    p.require_signed = true
-    p.include_gemspec = false
-    p.include_rakefile = true      
+  echoe = Echoe.new(GEM_NAME, VERS) do |p|
+    p.author = AUTHOR 
+    p.rubyforge_name = RUBYFORGE_NAME
+    p.name = NAME
+    p.description = DESCRIPTION
+    p.changes = CHANGES
+    p.email = EMAIL
+    p.summary = DESCRIPTION
+    p.url = HOMEPATH
+    p.need_tar = false
+    p.need_tar_gz = true
+    p.test_globs = ["*_test.rb"]
+    p.clean_globs = CLEAN  
+    p.spec_extras = {:post_install_message => 
+"
+Thanks for installing Bleak House #{VERS}. 
+
+For each Rails app you want to profile, you will need to add the following 
+rake task in RAILS_ROOT/lib/tasks/bleak_house_tasks.rake to be able to run 
+the analyzer: 
+" + taskmsg.join("  ") + "\n"}
   end
-         
-rescue LoadError
+            
+rescue LoadError => boom
+  puts "You are missing a dependency required for meta-operations on this gem."
+  puts "#{boom.to_s.capitalize}."
+  
+  desc 'Run the default tasks'
+  task :default => :test
 end
 
-desc 'Run tests.'
+desc 'Do nothing.'
 Rake::Task.redefine_task("test") do
-  system "ruby-bleak-house -Ibin:lib:test test/unit/test_bleak_house.rb #{ENV['METHOD'] ? "--name=#{ENV['METHOD']}" : ""}"
+   puts "There are no tests. You could totally write some, though."
+#   system "ruby -Ibin:lib:test test/unit/polymorph_test.rb #{ENV['METHOD'] ? "--name=#{ENV['METHOD']}" : ""}"
 end
-
-desc 'Build a patched binary.'
-namespace :ruby do
-  task :build do    
-    if RUBY_PLATFORM =~ /win32|windows/
-      puts "ERROR: windows not supported."
-      exit
-    end
-    require 'fileutils'
-    puts "building patched Ruby binary"
-    tmp = "/tmp/"
-    Dir.chdir(tmp) do
-      build_dir = "bleak_house"
-      binary_dir = File.dirname(`which ruby`)
-      FileUtils.rm_rf(build_dir) rescue nil
-      Dir.mkdir(build_dir)
-      begin
-        Dir.chdir(build_dir) do
-          puts "  downloading Ruby source"
-          bz2 = "ruby-1.8.6.tar.bz2"
-          system("wget 'http://rubyforge.org/frs/download.php/18434/ruby-1.8.6.tar.bz2' &> wget.log")
-          puts "  extracting"
-          system("tar xjf #{bz2} &> tar.log")
-          File.delete bz2
-          Dir.chdir("ruby-1.8.6") do
-            puts "  patching"
-            system("patch -p0 < \'#{File.dirname(__FILE__)}/patches/gc.c.patch\' &> ../gc.c.patch.log")
-            system("patch -p0 < \'#{File.dirname(__FILE__)}/patches/parse.y.patch\' &> ../parse.y.patch.log")
-            puts "  configuring"
-            system("./configure --prefix=#{binary_dir[0..-5]} &> ../configure.log") # --with-static-linked-ext
-            puts "  making"
-            system("make &> ../make.log")
-  
-            binary = "#{binary_dir}/ruby-bleak-house"
-            puts "  installing as #{binary}"
-            FileUtils.cp("./ruby", binary)
-            FileUtils.chmod(0755, binary)
-            puts "  done"
-          end
-        end
-      rescue Object => e
-        puts "ERROR: please see the last modified log file in #{tmp}#{build_dir}, perhaps\nit will contain a clue."
-      end    
-    end    
-  end
-end
-

data/init.rb

@@ -1,3 +1,2 @@
 
-puts "Please use the gem version of BleakHouse from now on."
-exit!
+require 'bleak_house'

data/install.rb

@@ -1,2 +1 @@
-
-puts "Please use the gem version of BleakHouse from now on."
+puts File.open("#{File.dirname(__FILE__)}/README").read

data/lib/bleak_house/{rails/action_controller.rb → action_controller.rb}

@@ -1,15 +1,14 @@
 
-# Override ActionController::Base.process and process_with_exception to make sure the request tag for the snapshot gets set as a side-effect of request processing.
 class ActionController::Base
   class << self
     def process_with_bleak_house(request, *args) 
-      BleakHouse::Rails.set_request_name request
+      BleakHouse.set_request_name request
       process_without_bleak_house(request, *args)
     end
     alias_method_chain :process, :bleak_house
     
     def process_with_exception_with_bleak_house(request, *args)
-      BleakHouse::Rails.set_request_name request, "/error"
+      BleakHouse.set_request_name request, "/error"
       process_with_exception_without_bleak_house(request, *args)
     end
     alias_method_chain :process_with_exception, :bleak_house    

data/lib/bleak_house/analyze.rb

@@ -0,0 +1,139 @@
+
+require 'rubygems'
+require 'fileutils'
+require 'base64'
+
+gem 'gruff', '= 0.2.8'
+require 'gruff'
+
+# require, but make rdoc not whine
+load "#{File.dirname(__FILE__)}/gruff_hacks.rb"
+load "#{File.dirname(__FILE__)}/support_methods.rb"
+
+Gruff::Base::LEFT_MARGIN = 200
+Gruff::Base::NEGATIVE_TOP_MARGIN = 30
+Gruff::Base::MAX_LEGENDS = 28
+
+class BleakHouse
+  class Analyze    
+    
+    DIR = "#{RAILS_ROOT}/log/bleak_house/"
+    
+    def initialize(data, increments, name)
+      @data = data
+      @increments = increments
+      @name = name
+    end 
+    
+    def draw    
+      g = Gruff::Line.new("1024x768")
+      g.title = @name
+      g.x_axis_label = "time"
+      g.legend_font_size = g.legend_box_size = 14
+      g.title_font_size = 24
+      g.marker_font_size = 14
+            
+      @data.map do |key, values|
+        name = if key.to_s == ""
+          '[Unknown]' 
+        else
+#        elsif key =~ Regexp.new(@specials.keys.join('|'))
+#          name = "#{key} (#{values.to_i.max / (2**10)} MB)"
+#          @specials.each do |regex, scale|
+#            values.map! {|x| x * scale} if key =~ regex
+#          end              
+#        else
+          "#{key.gsub(/.*::/, '')} (#{values.to_i.max})"
+        end
+        [name, values]
+      end.sort_by do |key, values|
+        0 - key[/.*?([\d]+)\)$/, 1].to_i
+      end.each do |key, values|
+        g.data(key, values.to_i)
+      end
+      
+      labels = {}
+      mod = (@increments.size / 4.0).ceil
+      @increments.each_with_index do |increment, index|
+        labels[index] = increment.split(" ").last if (index % mod).zero?
+      end
+      g.labels = labels
+      
+      g.minimum_value = 0
+#      g.maximum_value = @maximum
+      
+      g.write(@name.to_filename + ".png")
+    end
+    
+    def self.aggregate(data, selector, namer)
+      aggregate_data = {}
+      increments = []
+      data.each_with_index do |frameset, index|
+        increments << frameset.time
+        frameset.data.keys.select do |key| 
+          key =~ selector #or key =~ Regexp.new(specials.keys.join('|'))
+        end.each do |key|
+          aggregate_data[key.to_s[namer, 1]] ||= []
+          aggregate_data[key.to_s[namer, 1]][index] += frameset.data[key].to_i
+        end
+      end
+      [aggregate_data, increments]
+    end
+    
+    def self.build_all(filename)
+      unless File.exists? filename
+        puts "No data file found: #{filename}"
+        exit 
+      end
+      FileUtils.rm_r(DIR) rescue nil
+      Dir.mkdir(DIR)
+
+      Dir.chdir(DIR) do        
+
+        puts "Parsing data"
+        data = File.open(filename).readlines.map do |line|
+          Marshal.load Base64.decode64(line)
+        end
+        
+#        data_maximum = data.flatten.inject(0) do |max, el|
+#          if el.is_a? Hash
+#            current_max = el.merge({:"real memory" => 0, :"virtual memory" => 0}).values.to_i.max
+#            current_max if max < current_max
+#          end or max
+#        end
+#        mem_maximum = data.flatten.inject(0) do |max, el| # only real memory (RSS) for now
+#          (el["real memory"] if el.is_a?(Hash) and max < el["real memory"]) or max
+#        end
+#        mem_scale = data_maximum / mem_maximum.to_f
+#        scales = {/memory$/ => mem_scale}
+        
+        puts "By controller"
+        controller_data, increments = aggregate(data, //, /^(.*?)($|\/|::::)/)
+        Analyze.new(controller_data, increments, "objects by controller").draw
+                
+        # in each controller, by action
+        puts "By action"
+        controller_data.keys.each do |controller|
+          puts "  ...in #{controller} controller"
+          Dir.descend(controller) do             
+            action_data, increments = aggregate(data, /^#{controller}($|\/|::::)/, /\/(.*?)($|\/|::::)/)
+            Analyze.new(action_data, increments, "objects by action in /#{controller}").draw
+          
+          # in each action, by object class
+          action_data.keys.each do |action|
+            action = "unknown" if action.to_s == ""
+            puts "    ...in #{action} action"
+            Dir.descend(action) do
+              class_data, increments = aggregate(data, /^#{controller}#{"\/#{action}" unless action == "unknown"}($|\/|::::)/, 
+                /::::(.*)/)
+              Analyze.new(class_data, increments, "objects by class in /#{controller}/#{action}").draw
+            end
+          end
+          
+          end          
+        end
+      end    
+    end 
+    
+  end
+end

data/lib/bleak_house/bleak_house.rb

@@ -0,0 +1,33 @@
+
+class BleakHouse  
+  cattr_accessor :last_request_name
+  cattr_accessor :dispatch_count
+  cattr_accessor :log_interval
+
+  self.dispatch_count = 0
+  self.log_interval = (ENV['INTERVAL'] and ENV['INTERVAL'].to_i or 10)
+
+  def self.set_request_name request, other = nil
+    self.last_request_name = "#{request.parameters['controller']}/#{request.parameters['action']}/#{request.request_method}#{other}"
+  end
+
+  def self.debug s
+    s = "#{name.underscore}: #{s}"
+    ::ActiveRecord::Base.logger.debug s if ::ActiveRecord::Base.logger
+  end
+    
+  def self.warn s
+    s = "#{name.underscore}: #{s}"
+    if ::ActiveRecord::Base.logger
+      ::ActiveRecord::Base.logger.warn s
+    else
+      $stderr.puts s
+    end
+  end
+
+  LOGFILE = "#{RAILS_ROOT}/log/bleak_house_#{RAILS_ENV}.dump"
+  if File.exists?(LOGFILE)
+    File.rename(LOGFILE, "#{LOGFILE}.old") 
+    warn "renamed old logfile"
+  end
+end

data/lib/bleak_house/dispatcher.rb

@@ -0,0 +1,23 @@
+
+# crazyness
+
+class Dispatcher
+  class << self
+    def prepare_application_with_bleak_house
+      prepare_application_without_bleak_house
+      BleakHouse.dispatch_count += 1
+      BleakHouse::MemLogger.snapshot('core rails')
+    end
+    alias_method_chain :prepare_application, :bleak_house
+    
+    def reset_after_dispatch_with_bleak_house
+      BleakHouse::MemLogger.snapshot(BleakHouse.last_request_name || 'unknown')
+      if (BleakHouse.dispatch_count % BleakHouse.log_interval).zero?
+        BleakHouse.warn "wrote frameset (#{BleakHouse.dispatch_count} dispatches)"
+        BleakHouse::MemLogger.log(BleakHouse::LOGFILE)
+      end
+      reset_after_dispatch_without_bleak_house
+    end
+    alias_method_chain :reset_after_dispatch, :bleak_house
+  end
+end

data/lib/bleak_house/gruff_hacks.rb

@@ -0,0 +1,56 @@
+
+class Gruff::Base
+  
+  def draw_legend
+    @legend_labels = @data.collect {|item| item[DATA_LABEL_INDEX] }
+
+    legend_square_width = @legend_box_size # small square with color of this item
+    legend_left = 10
+
+    current_x_offset = legend_left
+    current_y_offset =  TOP_MARGIN + TITLE_MARGIN + @title_caps_height + LEGEND_MARGIN
+
+    debug { @d.line 0.0, current_y_offset, @raw_columns, current_y_offset }
+                                                  
+    @legend_labels.each_with_index do |legend_label, index|      
+
+      next if index > MAX_LEGENDS
+      legend_label = "Some not shown" if index == MAX_LEGENDS
+
+      # Draw label
+      @d.fill = @font_color
+      @d.font = @font if @font
+      @d.pointsize = scale_fontsize(@legend_font_size)
+      @d.stroke('transparent')
+      @d.font_weight = NormalWeight
+      @d.gravity = WestGravity
+      @d = @d.annotate_scaled( @base_image, 
+                        @raw_columns, 1.0,
+                        current_x_offset + (legend_square_width * 1.7), current_y_offset, 
+                        legend_label.to_s, @scale)
+      
+      if index < MAX_LEGENDS
+      # Now draw box with color of this dataset
+        @d = @d.stroke('transparent')
+        @d = @d.fill @data[index][DATA_COLOR_INDEX]
+        @d = @d.rectangle(current_x_offset, 
+                          current_y_offset - legend_square_width / 2.0, 
+                          current_x_offset + legend_square_width, 
+                          current_y_offset + legend_square_width / 2.0)
+      end
+
+      @d.pointsize = @legend_font_size
+      metrics = @d.get_type_metrics(@base_image, legend_label.to_s)
+      current_y_offset += metrics.height * 1.05
+    end
+    @color_index = 0
+  end
+  
+  alias :setup_graph_measurements_without_top_margin :setup_graph_measurements
+  def setup_graph_measurements  
+    setup_graph_measurements_without_top_margin
+    @graph_height += NEGATIVE_TOP_MARGIN
+    @graph_top -= NEGATIVE_TOP_MARGIN    
+  end
+  
+end

data/lib/bleak_house/mem_logger.rb

@@ -0,0 +1,54 @@
+
+require 'base64'
+
+class BleakHouse::MemLogger
+  class << self
+
+    SEEN = {}
+    CURRENT = {}
+    TAGS = Hash.new(0)
+    TIMEFORMAT = '%Y-%m-%d %H:%M:%S'
+    VSZ = "virtual memory"
+    RSS = "real memory"
+
+    def log(path, with_mem = false)
+      File.open(path, 'a+') do |log|
+        log.sync = true
+        TAGS[VSZ], TAGS[RSS] = mem_usage if with_mem
+        log.puts Base64.encode64(Marshal.dump([Time.now.strftime(TIMEFORMAT), TAGS])).gsub("\n", '')
+      end
+      GC.start
+    end
+    
+    def snapshot(tag)
+#      puts "bleak_house: seen: #{SEEN.size}"
+      CURRENT.clear
+      ObjectSpace.each_object do |obj|
+        CURRENT[obj_id = obj._bleak_house_object_id] = true
+        unless SEEN[obj_id]
+          # symbols will rapidly stabilize; don't worry
+          SEEN[obj_id] = "#{tag}::::#{obj._bleak_house_class}".to_sym 
+          TAGS[SEEN[obj_id]] += 1
+        end
+      end
+#      puts "bleak_house: current: #{CURRENT.size}"
+      SEEN.keys.each do |obj_id|
+        TAGS[SEEN.delete(obj_id)] -= 1 unless CURRENT[obj_id]          
+      end
+ #     puts "bleak_house: done"
+      CURRENT.clear
+      GC.start
+    end        
+    
+    def mem_usage
+      `ps -x -o 'rss vsz' -p #{Process.pid}`.split(/\s+/)[-2..-1].map{|el| el.to_i}
+    end
+    
+  end
+end
+
+class Object
+  alias :_bleak_house_object_id :object_id
+  alias :_bleak_house_class  :class
+end
+

data/lib/bleak_house/{support/core_extensions.rb → support_methods.rb}

@@ -1,6 +1,4 @@
 
-require 'set'
-
 class Array
   alias :time :first
   alias :data :last
@@ -15,24 +13,6 @@ class Array
     
 end
 
-class Hash
-
-  # Similar to the ActiveSupport methods in Rails
-  def slice(keys)
-    keys = Set.new(keys)
-    reject do |key,| 
-      !keys.include?(key)
-    end
-  end  
-  
-  def unslice(keys)
-    keys = Set.new(keys)
-    reject do |key,| 
-      keys.include?(key)
-    end
-  end
-end
-
 class Dir
   def self.descend path, &block
     path = path.split("/") unless path.is_a? Array
@@ -64,7 +44,4 @@ class Symbol
   def =~ regex
     self.to_s =~ regex
   end
-  def [](*args)
-    self.to_s[*args]
-  end
 end