hamsterdam 1.0.2 → 1.0.5

data/CHANGELOG

@@ -1,3 +1,8 @@
+Hamsterdam v1.0.5
+  * Setters return the same instance of the object if the value being set is unchanged
+  * Optimized creation of a new struct caused by a setter modification by skipping validation
+	* (v1.0.3 and 1.0.4 skipped because of github repo tagging conflict)
+
 Hamsterdam v1.0.2
   * Field name defs, constructor keys and merge keys are now stringified internally to prevent string/symbol mismatching
   * Added syntax helper for this kind of thing: class Castle < Hamsterdam::Struct(:gate, :walls)

data/lib/hamsterdam.rb

@@ -1,7 +1,6 @@
 require 'hamster'
 
 module Hamsterdam
-  VERSION = "1.0.2"
 
   def self.Struct(*field_names)
     Hamsterdam::Struct.define(*field_names)
@@ -13,11 +12,15 @@ module Hamsterdam
         field_names = field_names.map &:to_sym
         field_names.each do |fname|
           define_method fname do 
-            return @data[fname]
+            @data[fname]
           end
 
           define_method "set_#{fname}" do |value|
-            self.class.new(@data.put(fname, value))
+            if @data[fname] == value
+              self
+            else
+              self.class.new(@data.put(fname, value), false)
+            end
           end
         end
 
@@ -44,9 +47,13 @@ module Hamsterdam
       struct_class
     end
 
-    def initialize(values=Hamster.hash)
-      @data = flesh_out(ensure_hamster_hash(values))
-      validate_keys(@data)
+    def initialize(values=Hamster.hash, validate=true)
+      if validate
+        @data = flesh_out(ensure_hamster_hash(values))
+        validate_keys(@data)
+      else
+        @data = values
+      end
     end
 
     def merge(values)
@@ -74,7 +81,7 @@ module Hamsterdam
     end
 
     def to_s
-      name = self.class.name.split(/::/).last
+      name = self.class.name ? self.class.name.split(/::/).last : self.class.to_s
       data = to_hamster_hash
       fields = self.class.field_names_list.map { |fname| "#{fname}: #{data[fname].inspect}" }
       "<#{([name]+fields).join(" ")}>"

data/lib/hamsterdam/version.rb

@@ -0,0 +1,3 @@
+module Hamsterdam
+  VERSION = "1.0.5"
+end

data/spec/hamsterdam_spec.rb

@@ -121,6 +121,11 @@ describe "Hamsterdam structures" do
       struct.bottom.should == 1
     end
 
+    it "returns the same struct if a set does not change the value" do
+      struct = struct_class.new(top: 25, bottom: 2)
+      struct.set_top(25).set_bottom(2).should equal(struct)
+    end
+
     it "provides a merge function" do
       struct = struct_class.new(top: 10, bottom: 1)
 
@@ -146,6 +151,7 @@ describe "Hamsterdam structures" do
       struct.top.should == 10
       struct.bottom.should == 1
     end
+
   end
 
   describe "inspect and to_s" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hamsterdam
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-01-18 00:00:00.000000000 Z
+date: 2013-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hamster
@@ -98,14 +98,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- CHANGELOG
-- Gemfile
 - README.md
-- Rakefile
-- TODO.txt
-- hamsterdam.gemspec
+- CHANGELOG
+- lib/hamsterdam/version.rb
 - lib/hamsterdam.rb
-- rake_tasks/rspec.rake
 - spec/hamsterdam_spec.rb
 - spec/spec_helper.rb
 - spec/support/hamsterdam_helpers.rb

data/Gemfile

@@ -1,4 +0,0 @@
-source 'https://rubygems.org'
-
-# Deps are in hamsterdam.gemspec
-gemspec

data/Rakefile

@@ -1,12 +0,0 @@
-#!/usr/bin/env rake
-require "bundler/gem_tasks"
-HERE = File.dirname(__FILE__)
-
-Dir[File.expand_path(HERE) + "/rake_tasks/*.rake"].each do |rake_file|
-  import rake_file
-end
-
-# desc 'Default: run specs and cucumber features'
-# task :default => [ "spec", "cuc:features" ]
-
-task :default => [ "spec" ]

data/TODO.txt

@@ -1,4 +0,0 @@
-- Add commas in to_s
-- .set_defaults({}) to provide default values for constructors
-- Want to use fun "Status < Hamsterdam::Struct(:one, :two)" but something is wrong with inheritance of field_names that blows up the flesh_out func.
-- FIXED: Symbol/string insensitivity (use symbols internally and when listing field names, but accept :foo and 'foo' when constructing and merging

data/hamsterdam.gemspec

@@ -1,24 +0,0 @@
-# -*- encoding: utf-8 -*-
-$LOAD_PATH << File.expand_path(File.dirname(__FILE__) + "/lib")
-require File.expand_path('../lib/hamsterdam', __FILE__)
-
-Gem::Specification.new do |gem|
-  gem.authors       = ["David Crosby"]
-  gem.email         = ["david.crosby@atomicobject.com"]
-  gem.description   = %q{Immutable Struct-like record structures based on Hamster.}
-  gem.summary       = %q{Immutable Struct-like record structures based on Hamster.}
-  gem.homepage      = "https://github.com/atomicobject/hamsterdam"
-
-  gem.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  gem.files         = `git ls-files`.split("\n") - [".gitignore", ".rspec", ".rvmrc", "NOTES.txt", "TODO"]
-  gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  gem.name          = "hamsterdam"
-  gem.require_paths = ["lib"]
-  gem.version       = Hamsterdam::VERSION
-
-  gem.add_dependency "hamster"
-  gem.add_development_dependency "rake"
-  gem.add_development_dependency "rspec"
-  gem.add_development_dependency "simplecov"
-  gem.add_development_dependency "pry"
-end

data/rake_tasks/rspec.rake

@@ -1,25 +0,0 @@
-begin
-require 'rspec/core/rake_task'
-rescue Exception => e
-  "RSpec is not available.  'spec' task will not be defined."
-end
-
-begin # protect from missing rspec
-  desc "Run specs"
-  RSpec::Core::RakeTask.new do |t|
-    if ENV["file"]
-      t.pattern = ENV["file"]
-    end
-    t.rspec_opts = "--color --format documentation" # --tty
-  end
-  
-  desc "Run individual spec"
-  task "spec:just" do
-    RSpec::Core::RakeTask.new("_tmp_rspec") do |t|
-      t.pattern = ENV["file"] || raise("Please supply 'file' argument")
-      t.rspec_opts = "--color"
-    end
-    Rake::Task["_tmp_rspec"].invoke
-  end
-rescue Exception => e
-end