sdp 0.2.5 → 0.2.6

data/ChangeLog.rdoc

@@ -1,3 +1,10 @@
+=== 0.2.6 / 2011-12-02
+
+* Bug Fixes:
+  * gh-19[https://github.com/turboladen/sdp/issues/19]: Created +Hash#keys_to_s+
+    that converts all SDP parse result attributes to be Strings instead of
+    Parslet::Slices.
+
 === 0.2.5 / 2011-12-01
 
 * Improvements:

data/lib/ext/hash_patch.rb

@@ -0,0 +1,23 @@
+class Hash
+  # Converts keys to Strings.  Useful for dealing with +SDP::Parser+ results
+  # (which subclasses +Parslet::Parser+), which returns +Parslet::Slice+ objects
+  # in place of +String+ objects, which isn't helpful in this case.
+  #
+  # @return [Hash] +self+, but with Strings instead of Parslet::Slices.
+  def keys_to_s
+    self.each do |k, v|
+      case v
+      when Hash
+        v.keys_to_s
+      when Array
+        v.each do |element|
+          element.keys_to_s if element.is_a? Hash
+        end
+      else
+        self[k] = v.to_s
+      end
+    end
+
+    self
+  end
+end

data/lib/sdp.rb

@@ -2,6 +2,7 @@ require File.expand_path(File.dirname(__FILE__) + '/sdp/version')
 require File.expand_path(File.dirname(__FILE__) + '/sdp/description')
 require File.expand_path(File.dirname(__FILE__) + '/sdp/parser')
 require File.expand_path(File.dirname(__FILE__) + '/sdp/parse_error')
+require File.expand_path(File.dirname(__FILE__) + '/ext/hash_patch')
 
 # The only use for this class is the +#parse+ method, which is in this
 # base class solely for convenience.  Other than this method, this
@@ -23,6 +24,6 @@ class SDP
       raise SDP::ParseError, ex
     end
 
-    SDP::Description.new(sdp_hash)
+    SDP::Description.new(sdp_hash.keys_to_s)
   end
 end

data/lib/sdp/version.rb

@@ -1,3 +1,3 @@
 class SDP
-  VERSION = "0.2.5"
+  VERSION = "0.2.6"
 end

data/spec/sdp_spec.rb

@@ -23,8 +23,8 @@ a=rtpmap:99 h263-1998/90000
 EOF
 
 describe SDP do
-  it "VERSION is 0.2.5" do
-    SDP.const_get('VERSION').should eql '0.2.5'
+  it "VERSION is 0.2.6" do
+    SDP.const_get('VERSION').should eql '0.2.6'
   end
 
   it "raises when parsing malformed SDP text" do
@@ -42,133 +42,133 @@ describe SDP do
 
     it "has a version number of 0" do
       @parsed_sdp.protocol_version.should == "0"
-      @parsed_sdp.protocol_version.should be_a Parslet::Slice
+      @parsed_sdp.protocol_version.should be_a String
     end
 
     context "origin" do
       it "has a username of 'jdoe'" do
         @parsed_sdp.username.should == 'jdoe'
-        @parsed_sdp.username.should be_a Parslet::Slice
+        @parsed_sdp.username.should be_a String
       end
 
       it "has a session_id of '2890844526'" do
         @parsed_sdp.id.should == "2890844526"
-        @parsed_sdp.id.should be_a Parslet::Slice
+        @parsed_sdp.id.should be_a String
       end
 
       it "has a session_version of '2890842807'" do
         @parsed_sdp.version.should == '2890842807'
-        @parsed_sdp.version.should be_a Parslet::Slice
+        @parsed_sdp.version.should be_a String
       end
 
       it "has a net_type of 'IN'" do
         @parsed_sdp.network_type.should == "IN"
-        @parsed_sdp.network_type.should be_a Parslet::Slice
+        @parsed_sdp.network_type.should be_a String
       end
 
       it "has a address_type of 'IP4'" do
         @parsed_sdp.address_type.should == "IP4"
-        @parsed_sdp.address_type.should be_a Parslet::Slice
+        @parsed_sdp.address_type.should be_a String
       end
 
       it "has a unicast_address of '10.47.16.5'" do
         @parsed_sdp.unicast_address.should == "10.47.16.5"
-        @parsed_sdp.unicast_address.should be_a Parslet::Slice
+        @parsed_sdp.unicast_address.should be_a String
       end
     end
 
     it "has a session name of 'SDP Seminar'" do
       @parsed_sdp.name.should == "SDP Seminar"
-      @parsed_sdp.name.should be_a Parslet::Slice
+      @parsed_sdp.name.should be_a String
     end
 
     it "has a session information of 'A Seminar on the session description protocol'" do
       @parsed_sdp.information.should == "A Seminar on the session description protocol"
-      @parsed_sdp.information.should be_a Parslet::Slice
+      @parsed_sdp.information.should be_a String
     end
 
     it "has a URI of 'http://www.example.com/seminars/sdp.pdf'" do
       @parsed_sdp.uri.should == "http://www.example.com/seminars/sdp.pdf"
-      @parsed_sdp.uri.should be_a Parslet::Slice
+      @parsed_sdp.uri.should be_a String
     end
 
     it "has an email address of 'j.doe@example.com (Jane Doe)'" do
       @parsed_sdp.email_address.should == "j.doe@example.com (Jane Doe)"
-      @parsed_sdp.email_address.should be_a Parslet::Slice
+      @parsed_sdp.email_address.should be_a String
     end
 
     it "has a phone number of '+1 617 555-6011'" do
       @parsed_sdp.phone_number.should == "+1 617 555-6011"
-      @parsed_sdp.phone_number.should be_a Parslet::Slice
+      @parsed_sdp.phone_number.should be_a String
     end
 
     context "bandwidth" do
       it "has a bandwidth type of 'CT'" do
         @parsed_sdp.bandwidth_type.should == "CT"
-        @parsed_sdp.bandwidth_type.should be_a Parslet::Slice
+        @parsed_sdp.bandwidth_type.should be_a String
       end
 
       it "has a bandwidth of '1000'" do
         @parsed_sdp.bandwidth.should == "1000"
-        @parsed_sdp.bandwidth.should be_a Parslet::Slice
+        @parsed_sdp.bandwidth.should be_a String
       end
     end
 
     context "timing" do
       it "has a start time of '2873397496'" do
         @parsed_sdp.start_time.should == '2873397496'
-        @parsed_sdp.start_time.should be_a Parslet::Slice
+        @parsed_sdp.start_time.should be_a String
       end
 
       it "has a stop time of '2873404696'" do
         @parsed_sdp.stop_time.should == '2873404696'
-        @parsed_sdp.stop_time.should be_a Parslet::Slice
+        @parsed_sdp.stop_time.should be_a String
       end
     end
 
     context "repeat times" do
       it "has a repeat interval of '604800'" do
         @parsed_sdp.repeat_interval.should == '604800'
-        @parsed_sdp.repeat_interval.should be_a Parslet::Slice
+        @parsed_sdp.repeat_interval.should be_a String
       end
 
       it "has an active duration of '3600'" do
         @parsed_sdp.active_duration.should == '3600'
-        @parsed_sdp.active_duration.should be_a Parslet::Slice
+        @parsed_sdp.active_duration.should be_a String
       end
 
       it "has a offsets from start time of '0 90000'" do
         @parsed_sdp.offsets_from_start_time.should == '0 90000'
-        @parsed_sdp.offsets_from_start_time.should be_a Parslet::Slice
+        @parsed_sdp.offsets_from_start_time.should be_a String
       end
     end
 
     context "time zones" do
       it "has a time zone adjustment of '2882844526'" do
         @parsed_sdp.time_zones[:adjustment_time].should == '2882844526'
-        @parsed_sdp.time_zones[:adjustment_time].should be_a Parslet::Slice
+        @parsed_sdp.time_zones[:adjustment_time].should be_a String
       end
 
       it "has a time zone offset of '-1h'" do
         @parsed_sdp.time_zones[:offset].should == '-1h'
-        @parsed_sdp.time_zones[:offset].should be_a Parslet::Slice
+        @parsed_sdp.time_zones[:offset].should be_a String
       end
     end
 
     context "connection data" do
       it "has a connection network type of 'IN'" do
         @parsed_sdp.connection_network_type.should == "IN"
-        @parsed_sdp.connection_network_type.should be_a Parslet::Slice
+        @parsed_sdp.connection_network_type.should be_a String
       end
 
       it "has a addrtype of :IP4" do
         @parsed_sdp.connection_address_type.should == "IP4"
-        @parsed_sdp.connection_address_type.should be_a Parslet::Slice
+        @parsed_sdp.connection_address_type.should be_a String
       end
 
       it "has a connection address of '224.2.17.12/127'" do
         @parsed_sdp.connection_address.should == '224.2.17.12/127'
-        @parsed_sdp.connection_address.should be_a Parslet::Slice
+        @parsed_sdp.connection_address.should be_a String
       end
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sdp
 version: !ruby/object:Gem::Version
-  version: 0.2.5
+  version: 0.2.6
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-12-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parslet
-  requirement: &70223149043040 !ruby/object:Gem::Requirement
+  requirement: &70098175145760 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 1.1.0
   type: :runtime
   prerelease: false
-  version_requirements: *70223149043040
+  version_requirements: *70098175145760
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70223149042480 !ruby/object:Gem::Requirement
+  requirement: &70098175145200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>'
@@ -32,10 +32,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *70223149042480
+  version_requirements: *70098175145200
 - !ruby/object:Gem::Dependency
   name: code_statistics
-  requirement: &70223149041980 !ruby/object:Gem::Requirement
+  requirement: &70098175144700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 0.2.13
   type: :development
   prerelease: false
-  version_requirements: *70223149041980
+  version_requirements: *70098175144700
 - !ruby/object:Gem::Dependency
   name: cucumber
-  requirement: &70223149041460 !ruby/object:Gem::Requirement
+  requirement: &70098175144180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,10 +54,10 @@ dependencies:
         version: 1.1.0
   type: :development
   prerelease: false
-  version_requirements: *70223149041460
+  version_requirements: *70098175144180
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70223149040940 !ruby/object:Gem::Requirement
+  requirement: &70098175143660 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70223149040940
+  version_requirements: *70098175143660
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70223149040440 !ruby/object:Gem::Requirement
+  requirement: &70098175143140 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: 2.6.0
   type: :development
   prerelease: false
-  version_requirements: *70223149040440
+  version_requirements: *70098175143140
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &70223149039920 !ruby/object:Gem::Requirement
+  requirement: &70098175142640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: 0.5.0
   type: :development
   prerelease: false
-  version_requirements: *70223149039920
+  version_requirements: *70098175142640
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &70223149039400 !ruby/object:Gem::Requirement
+  requirement: &70098175142120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,7 +98,7 @@ dependencies:
         version: 0.6.0
   type: :development
   prerelease: false
-  version_requirements: *70223149039400
+  version_requirements: *70098175142120
 description: ! "This gem allows for parsing SDP (Session Description\n    Protocol)
   information in to a Ruby object, making it easy to read \n    and work with that
   data.  It also allows for easily creating SDP objects \n    that can be converted
@@ -119,6 +119,7 @@ files:
 - features/step_definitions/sdp_get_steps.rb
 - features/support/env.rb
 - features/support/sdp_file.txt
+- lib/ext/hash_patch.rb
 - lib/sdp/description.rb
 - lib/sdp/parse_error.rb
 - lib/sdp/parser.rb