ruby-protocol-buffers 1.0.1 → 1.1.0

data/VERSION

@@ -1 +1 @@
-1.0.1
+1.1.0

data/lib/protocol_buffers/compiler/file_descriptor_to_ruby.rb

@@ -29,10 +29,6 @@ HEADER
     @io.write("\n") unless descriptor.dependency.empty?
 
     # in_namespace correctly handles the case where @package.nil?
-    unless @package.empty?
-      @io.write("# Reload support\nObject.__send__(:remove_const, :#{@package}) if defined?(#{@package})\n\n")
-    end
-
     in_namespace("module", @package) do
       declare(descriptor.message_type, descriptor.enum_type)
 

data/lib/protocol_buffers/runtime/message.rb

@@ -213,9 +213,8 @@ module ProtocolBuffers
   #   end
   #
   # An exception will be thrown if an enum field is assigned a value not in the
-  # enum. Right now, this includes throwing an exception while parsing. This may
-  # change in the future to match the C++ behavior of treating it as an unknown
-  # tag number.
+  # enum. If an unknown enum value is found while parsing a message, this is
+  # treated like an unknown tag id. This matches the C++ library behavior.
   #
   # == Extensions
   #

data/spec/runtime_spec.rb

@@ -8,6 +8,14 @@ require 'protocol_buffers/compiler'
 
 describe ProtocolBuffers, "runtime" do
   before(:each) do
+    # clear our namespaces
+    Object.send(:remove_const, :Simple) if defined?(Simple)
+    Object.send(:remove_const, :Featureful) if defined?(Featureful)
+    Object.send(:remove_const, :Foo) if defined?(Foo)
+    Object.send(:remove_const, :TehUnknown) if defined?(TehUnknown)
+    Object.send(:remove_const, :TehUnknown2) if defined?(TehUnknown2)
+    Object.send(:remove_const, :TehUnknown3) if defined?(TehUnknown3)
+
     ProtocolBuffers::Compiler.compile_and_load(
       File.join(File.dirname(__FILE__), "proto_files", "simple.proto"))
     ProtocolBuffers::Compiler.compile_and_load(
@@ -341,14 +349,14 @@ describe ProtocolBuffers, "runtime" do
 
     # now make field_1 required
     ProtocolBuffers::Compiler.compile_and_load_string <<-EOS
-      package tehUnknown;
+      package tehUnknown2;
       message MyResult {
         required string field_1 = 1;
         optional string field_2 = 2;
       }
     EOS
 
-    proc { TehUnknown::MyResult.parse(buf) }.should raise_error(ProtocolBuffers::DecodeError)
+    proc { TehUnknown2::MyResult.parse(buf) }.should raise_error(ProtocolBuffers::DecodeError)
   end
 
   it "enforces valid values on deserialization" do
@@ -363,13 +371,13 @@ describe ProtocolBuffers, "runtime" do
     buf = res1.to_s
 
     ProtocolBuffers::Compiler.compile_and_load_string <<-EOS
-      package tehUnknown;
+      package tehUnknown2;
       message MyResult {
         optional int32 field_1 = 1;
       }
     EOS
 
-    proc { TehUnknown::MyResult.parse(buf) }.should raise_error(ProtocolBuffers::DecodeError)
+    proc { TehUnknown2::MyResult.parse(buf) }.should raise_error(ProtocolBuffers::DecodeError)
   end
 
   it "ignores and passes on unknown fields" do
@@ -389,7 +397,7 @@ describe ProtocolBuffers, "runtime" do
 
     # remove field_2 to pretend we never knew about it
     ProtocolBuffers::Compiler.compile_and_load_string <<-EOS
-      package tehUnknown;
+      package tehUnknown2;
       message MyResult {
         optional int32 field_1 = 1;
         optional int32 field_3 = 3;
@@ -398,7 +406,7 @@ describe ProtocolBuffers, "runtime" do
 
     res2 = nil
     proc do
-      res2 = TehUnknown::MyResult.parse(serialized)
+      res2 = TehUnknown2::MyResult.parse(serialized)
     end.should_not raise_error()
 
     res2.field_1.should == 0xffff
@@ -412,7 +420,7 @@ describe ProtocolBuffers, "runtime" do
 
     # now we know about field_2 again
     ProtocolBuffers::Compiler.compile_and_load_string <<-EOS
-      package tehUnknown;
+      package tehUnknown3;
       message MyResult {
         optional int32 field_1 = 1;
         optional int32 field_2 = 2;
@@ -420,7 +428,7 @@ describe ProtocolBuffers, "runtime" do
       }
     EOS
 
-    res3 = TehUnknown::MyResult.parse(serialized2)
+    res3 = TehUnknown3::MyResult.parse(serialized2)
     res3.field_1.should == 0xffff
 
     res3.field_2.should == 0xfffe
@@ -444,7 +452,7 @@ describe ProtocolBuffers, "runtime" do
 
     # remove field_2 to pretend we never knew about it
     ProtocolBuffers::Compiler.compile_and_load_string <<-EOS
-      package tehUnknown;
+      package tehUnknown2;
       message MyResult {
         enum E {
           V1 = 1;
@@ -455,7 +463,7 @@ describe ProtocolBuffers, "runtime" do
 
     res2 = nil
     proc do
-      res2 = TehUnknown::MyResult.parse(serialized)
+      res2 = TehUnknown2::MyResult.parse(serialized)
     end.should_not raise_error()
 
     res2.value_for_tag?(1).should be_false
@@ -465,7 +473,7 @@ describe ProtocolBuffers, "runtime" do
 
     # now we know about field_2 again
     ProtocolBuffers::Compiler.compile_and_load_string <<-EOS
-      package tehUnknown;
+      package tehUnknown3;
       message MyResult {
         enum E {
           V1 = 1;
@@ -475,7 +483,7 @@ describe ProtocolBuffers, "runtime" do
       }
     EOS
 
-    res3 = TehUnknown::MyResult.parse(serialized2)
+    res3 = TehUnknown3::MyResult.parse(serialized2)
     res3.field_1.should == 2
   end
 

metadata

@@ -1,104 +1,80 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: ruby-protocol-buffers
-version: !ruby/object:Gem::Version 
-  hash: 21
+version: !ruby/object:Gem::Version
+  version: 1.1.0
   prerelease: 
-  segments: 
-  - 1
-  - 0
-  - 1
-  version: 1.0.1
 platform: ruby
-authors: 
+authors:
 - Brian Palmer
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2011-03-20 00:00:00 -06:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2011-09-20 00:00:00.000000000Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: rspec
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  requirement: &70109784526400 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 9
-        segments: 
-        - 2
-        - 5
-        version: "2.5"
+      - !ruby/object:Gem::Version
+        version: '2.5'
   type: :development
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: autotest-standalone
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  version_requirements: *70109784526400
+- !ruby/object:Gem::Dependency
+  name: autotest-standalone
+  requirement: &70109784526000 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: autotest-growl
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  version_requirements: *70109784526000
+- !ruby/object:Gem::Dependency
+  name: autotest-growl
+  requirement: &70109784525540 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: rcov
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  version_requirements: *70109784525540
+- !ruby/object:Gem::Dependency
+  name: rcov
+  requirement: &70109784525120 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
-  name: yard
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  version_requirements: *70109784525120
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: &70109784524700 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id005
+  prerelease: false
+  version_requirements: *70109784524700
 description: 
-email: 
+email:
 - brian@codekitchen.net
-executables: 
+executables:
 - ruby-protoc
 extensions: []
-
-extra_rdoc_files: 
+extra_rdoc_files:
 - Changelog.md
-files: 
+files:
 - .document
 - .gitignore
 - Changelog.md
@@ -143,41 +119,31 @@ files:
 - spec/runtime_spec.rb
 - spec/spec.opts
 - spec/spec_helper.rb
-has_rdoc: true
 homepage: https://github.com/mozy/ruby-protocol-buffers
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.6.1
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: Ruby compiler and runtime for the google protocol buffers library.
-test_files: 
+test_files:
 - spec/compiler_spec.rb
 - spec/fields_spec.rb
 - spec/proto_files/depends.proto