gir_ffi 0.9.1 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a091e4dbd8360e158b2fb09d53f34fa6913dda07
-  data.tar.gz: 98c7a541d76f2e6217dca26d8cfd2ed5d7b32c71
+  metadata.gz: ca3636ea5b78e4f09c583690ad160c4a2e739cb1
+  data.tar.gz: 8a4d243567cc87cc00d18783931c97a35635c6c1
 SHA512:
-  metadata.gz: 1aaf7d901357faa26299c194deedb7a6cd42dafd62bb50e5f30c0c364ce4ef25349381deac8a4da8e42aa0d26d6d6bb2f0d1019884bf9c92a55fa03770df6485
-  data.tar.gz: cfd6c0d21b4f674c1fd637f4d8d963158f29264e046204021fb539594a1015f0807c279047c998b71fca4bc8f53ca2d750edd433af3800fab01e6727ec548ccd
+  metadata.gz: 6994483aa5dd9df51ef778c97de12698abf57fa436527370c404e7e53e711f5febbe56bd34d1765e2efed09622d81ffbb3da8bb45d3b68048956a61d45562d68
+  data.tar.gz: 0092e6fb91d2cfa3e1f6f70e4d601b8a1ac617d817675f3c1ad5930c0b3e0989c5b93a3b56601b2e6f1939c15cb1016485baff82635781af01e5099936232df8

data/Changelog.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.9.2 / 2016-02-05
+
+* Do not create a property accessor when a corresponding getter method exists
+
 ## 0.9.1 / 2016-02-04
 
 * Add field accessors for Object types for fields that don't have corresponding

data/lib/gir_ffi/builders/object_builder.rb

@@ -45,6 +45,12 @@ module GirFFI
         end
       end
 
+      def eligible_properties
+        info.properties.reject do |pinfo|
+          info.find_instance_method("get_#{pinfo.name}")
+        end
+      end
+
       protected
 
       def object_class_struct_info
@@ -102,7 +108,7 @@ module GirFFI
       end
 
       def setup_property_accessors
-        info.properties.each do |prop|
+        eligible_properties.each do |prop|
           PropertyBuilder.new(prop).build
         end
       end

data/lib/gir_ffi/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 # Current GirFFI version
 module GirFFI
-  VERSION = '0.9.1'.freeze
+  VERSION = '0.9.2'.freeze
 end

data/test/gir_ffi/builders/object_builder_test.rb

@@ -99,4 +99,21 @@ describe GirFFI::Builders::ObjectBuilder do
       result.map(&:name).wont_include 'parent_instance'
     end
   end
+
+  describe '#eligible_properties' do
+    let(:wi_builder) do
+      GirFFI::Builders::ObjectBuilder.new(
+        get_introspection_data('Regress', 'TestWi8021x'))
+    end
+
+    it 'includes properties that do not have a matching getter method' do
+      result = obj_builder.eligible_properties
+      result.map(&:name).must_include 'double'
+    end
+
+    it 'skips properties that have a matching getter method' do
+      result = wi_builder.eligible_properties
+      result.map(&:name).wont_include 'testbool'
+    end
+  end
 end

data/test/integration/generated_gst_test.rb

@@ -14,5 +14,18 @@ describe 'the generated Gst module' do
       GObject.signal_emit(instance, 'handoff')
       a.must_equal 10
     end
+
+    it 'correctly fetches the name' do
+      instance.name.must_equal 'sink'
+    end
+  end
+
+  describe 'Gst::AutoAudioSink' do
+    let(:instance) { Gst::ElementFactory.make('autoaudiosink', 'audiosink') }
+
+    it 'correctly fetches the name' do
+      instance.get_name.must_equal 'audiosink'
+      instance.name.must_equal 'audiosink'
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gir_ffi
 version: !ruby/object:Gem::Version
-  version: 0.9.1
+  version: 0.9.2
 platform: ruby
 authors:
 - Matijs van Zuijlen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-04 00:00:00.000000000 Z
+date: 2016-02-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi