gir_ffi 0.8.1 → 0.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5e373464c8f376bf817deb6d9f1d82317b9d0b6a
-  data.tar.gz: 9205d3435def6eecba27339a6bd7d0f1a114c745
+  metadata.gz: 6121625e854dd754146516dab64b67c650155c67
+  data.tar.gz: e486edea5ebffe7146c68dca786796f94ac58945
 SHA512:
-  metadata.gz: 42daa42701febe109c324c616af3aee367be2382e6994ca7fc9b915d63ef2984daacaed84eb6e1d5dd84d125e2acc964beeacb49e465552f9b45ad57f66a676c
-  data.tar.gz: 5f88e3c9d6e23449d0b461aab4bb320322f338e40f2938f5dd2bd9408da976b941d7661ad762c2ebd218f20521adb6c34dc27deab1316724d4bb6e3de0601bc2
+  metadata.gz: 170a26c6f8aecc34a77722a5525d8b1534491febdd244fff0a9c88aa298a357ee0914f1fc15393da52b306d2f295e7aaba0180f258a187e942433658e51262ae
+  data.tar.gz: afbed9ce4c8869f8e44b6c1bbb36d22192bf27e01720874d9e3632fd5dd4b59fcc5604e785c39ccf1c6b8e6fe084270d7efee54f992a70e36dc469fd93508031

data/Changelog.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 0.8.2 / 2015-10-10
+
+* Use inherited constructor for boxed types
+* Make InOutPointer work correctly for boxed types
+  * Make .for work with boxed types
+  * Make #set_value work with boxed types
+* Make GObject::Value support nil type:
+  * Make .wrap_ruby_value(nil) work
+  * Make #set_value and #get_value work when the type is TYPE_INVALID
+  * Make .for_gtype work with TYPE_INVALID and TYPE_NONE
+  * Make .from(nil) return a GObject::Value instead of nil
+  * Make #set_ruby_value private
+* Make GObject::Object.signal_emit work with gobject-introspection 1.46
+* Replace or remove all custom .new methods
+* Make setup_method and setup_instance_method handle symbol arguments
+
 ## 0.8.1 / 2015-10-04
 
 * Handle struct array fields with related length fields

data/DESIGN.md

@@ -64,25 +64,27 @@ cannot bootstrap itself.
 
 ## Object initialization
 
-An attempt at making Thing.new less hacky.
+Each constructor method is implemented in Ruby by a pair of new/initialize
+methods. For example, a constructor `new_from_file` is implemented as a
+combination of the singleton method `new_from_file` and the instance method
+`initialize_from_file`. User-created subclasses override the appropriate
+initializer method and must call super with the appropriate arguments.
 
-Goals:
-
-* No aliasing of Ruby's new. Overriding is possible with super being called.
-* #initialize should behave as expected. We may enforce use of super in Ruby
-  subclasses.
-
-Schematic depiction of what happens (can happen):
+Here is an example of the generated pair of methods:
 
 ```ruby
-class GObject::Object
-  def self.new *args
-    # Stage A
-    super(*other_args)
-    # Stage C
+class Regress::TestObj
+  def self.new_from_file(*args)
+    obj = allocate
+    obj.__send__ :initialize_from_file, *args
+    obj
   end
 
-  def initialize *other_args
-    # Stage B
+  def initialize_from_file(x)
+    _v1 = GirFFI::InPointer.from(:utf8, x)
+    _v2 = FFI::MemoryPointer.new(:pointer).write_pointer nil
+    _v3 = Regress::Lib.regress_test_obj_new_from_file _v1, _v2
+    GirFFI::ArgHelper.check_error(_v2)
+    store_pointer(_v3)
   end
 end

data/lib/ffi-glib/array.rb

@@ -13,11 +13,10 @@ module GLib
 
     attr_reader :element_type
 
-    class << self; undef :new; end
-
-    def self.new(type)
-      ptr = Lib.g_array_new(0, 0, calculated_element_size(type))
-      wrap type, ptr
+    def initialize(type)
+      @element_type = type
+      ptr = Lib.g_array_new(0, 0, calculated_element_size)
+      store_pointer(ptr)
     end
 
     # @api private

data/lib/ffi-glib/byte_array.rb

@@ -13,10 +13,8 @@ module GLib
       self.class.wrap(Lib.g_byte_array_append to_ptr, bytes, len)
     end
 
-    class << self; undef :new; end
-
-    def self.new
-      wrap(Lib.g_byte_array_new)
+    def initialize
+      store_pointer(Lib.g_byte_array_new)
     end
   end
 end

data/lib/ffi-glib/bytes.rb

@@ -32,13 +32,23 @@ module GLib
       end
     end
 
+    # Why redefine new here? The class builder will only define the full
+    # version of 'new' on first call, causing both new and initialize to be
+    # defined. This overwrites the custom version of initialize below.
+    # TODO: Improve class builder so this trick is not needed.
     class << self
       undef new
     end
 
-    def self.new(arr)
+    def self.new(*args)
+      obj = allocate
+      obj.__send__ :initialize, *args
+      obj
+    end
+
+    def initialize(arr)
       data = GirFFI::SizedArray.from :guint8, arr.size, arr
-      wrap Lib.g_bytes_new data.to_ptr, data.size
+      store_pointer Lib.g_bytes_new data.to_ptr, data.size
     end
 
     private

data/lib/ffi-glib/hash_table.rb

@@ -11,11 +11,11 @@ module GLib
     attr_reader :key_type
     attr_reader :value_type
 
-    class << self; remove_method :new; end
-
-    def self.new(keytype, valtype)
-      wrap [keytype, valtype], Lib.g_hash_table_new(
-        hash_function_for(keytype), equality_function_for(keytype))
+    def initialize(key_type, value_type)
+      @key_type = key_type
+      @value_type = value_type
+      store_pointer Lib.g_hash_table_new(
+        hash_function_for_key_type, equality_function_for_key_type)
     end
 
     # @api private
@@ -27,32 +27,6 @@ module GLib
       ghash
     end
 
-    # @api private
-    def self.hash_function_for(keytype)
-      case keytype
-      when :utf8
-        FFI::Function.new(:uint,
-                          [:pointer],
-                          find_support_function('g_str_hash'))
-      end
-    end
-
-    # @api private
-    def self.equality_function_for(keytype)
-      case keytype
-      when :utf8
-        FFI::Function.new(:int,
-                          [:pointer, :pointer],
-                          find_support_function('g_str_equal'))
-      end
-    end
-
-    # @api private
-    def self.find_support_function(name)
-      lib = ::GLib::Lib.ffi_libraries.first
-      lib.find_function(name)
-    end
-
     def each
       prc = proc do|keyptr, valptr, _userdata|
         key = GirFFI::ArgHelper.cast_from_pointer key_type, keyptr
@@ -79,5 +53,31 @@ module GLib
       @key_type, @value_type = *typespec
       self
     end
+
+    private
+
+    def hash_function_for_key_type
+      case @key_type
+      when :utf8
+        FFI::Function.new(:uint,
+                          [:pointer],
+                          find_support_function('g_str_hash'))
+      end
+    end
+
+    def equality_function_for_key_type
+      case @key_type
+      when :utf8
+        FFI::Function.new(:int,
+                          [:pointer, :pointer],
+                          find_support_function('g_str_equal'))
+      end
+    end
+
+    def find_support_function(name)
+      lib = ::GLib::Lib.ffi_libraries.first
+      lib.find_function(name)
+    end
+
   end
 end

data/lib/ffi-glib/list_methods.rb

@@ -11,9 +11,6 @@ module GLib
       replace_method base, :next, :tail
       replace_method base, :data, :head
 
-      class << base; self end.send :remove_method, :new
-      base.extend ListClassMethods
-
       base.extend ContainerClassMethods
     end
 
@@ -24,6 +21,11 @@ module GLib
       end
     end
 
+    def initialize(type)
+      store_pointer(FFI::Pointer.new(0))
+      @element_type = type
+    end
+
     def each
       reset_iterator
       while (elem = next_element)
@@ -64,17 +66,5 @@ module GLib
     def element_ptr_for(data)
       GirFFI::InPointer.from(element_type, data)
     end
-
-    # Common class methods for List and SList
-    module ListClassMethods
-      # TODO: Make this behave more like a real .new method
-      def new(type)
-        allocate.tap do |it|
-          struct = self::Struct.new(FFI::Pointer.new(0))
-          it.instance_variable_set :@struct, struct
-          it.instance_variable_set :@element_type, type
-        end
-      end
-    end
   end
 end

data/lib/ffi-glib/ptr_array.rb

@@ -16,13 +16,13 @@ module GLib
     POINTER_SIZE = FFI.type_size(:pointer)
 
     class << self
-      remove_method :new
       # Remove stub generated by builder.
       remove_method :add if method_defined? :add
     end
 
-    def self.new(type)
-      wrap(type, Lib.g_ptr_array_new)
+    def initialize(type)
+      @element_type = type
+      store_pointer Lib.g_ptr_array_new
     end
 
     def self.from_enumerable(type, it)

data/lib/ffi-gobject/closure.rb

@@ -5,6 +5,8 @@ module GObject
   #
   # To create Closure objects wrapping Ruby code, use {RubyClosure}.
   class Closure
+    setup_method :new_simple
+
     # @override
     #
     # @param [Proc] marshal The marshaller to use for this closure object

data/lib/ffi-gobject/ruby_closure.rb

@@ -15,14 +15,12 @@ module GObject
              :block_id, :int64
     end
 
-    def self.new(&block)
+    def initialize(&block)
       raise ArgumentError unless block_given?
 
-      closure = wrap(new_simple(self::Struct.size, nil).to_ptr)
-      closure.block = block
-      closure.set_marshal proc { |*args| marshaller(*args) }
-
-      closure
+      initialize_simple(self.class::Struct.size, nil)
+      self.block = block
+      set_marshal proc { |*args| self.class.marshaller(*args) }
     end
 
     # @api private
@@ -36,24 +34,24 @@ module GObject
 
       result = rclosure.invoke_block(*args)
 
-      return_value.set_ruby_value(result) if return_value
-    end
-
-    # @api private
-    # TODO: Re-structure so block= and invoke_block can become private methods
-    def block=(block)
-      id = block.object_id
-      BLOCK_STORE[id] = block
-      @struct[:block_id] = id
+      return_value.set_value(result) if return_value
     end
 
     # @api private
+    # TODO: Re-structure so invoke_block can become a private method
     def invoke_block(*args)
       block.call(*args)
     end
 
     private
 
+    # @api private
+    def block=(block)
+      id = block.object_id
+      BLOCK_STORE[id] = block
+      @struct[:block_id] = id
+    end
+
     def block
       BLOCK_STORE[@struct[:block_id]]
     end

data/lib/ffi-gobject/value.rb

@@ -6,6 +6,7 @@ module GObject
     setup_instance_method 'init'
 
     def init_with_finalizer(type)
+      return self if [TYPE_NONE, TYPE_INVALID].include? type
       init_without_finalizer(type).tap do
         ObjectSpace.define_finalizer self, self.class.make_finalizer(to_ptr)
       end
@@ -20,13 +21,8 @@ module GObject
       end
     end
 
-    # TODO: Give more generic name
-    def set_ruby_value(val)
-      init_for_ruby_value val if current_gtype == 0
-      set_value val
-    end
-
     METHOD_MAP = {
+      TYPE_INVALID => [:get_none,          :set_none],
       TYPE_BOOLEAN => [:get_boolean,       :set_boolean],
       TYPE_BOXED   => [:get_boxed,         :set_boxed],
       TYPE_CHAR    => [:get_char,          :set_char],
@@ -55,23 +51,6 @@ module GObject
 
     alias_method :value=, :set_value
 
-    CLASS_TO_GTYPE_MAP = {
-      TrueClass => TYPE_BOOLEAN,
-      FalseClass => TYPE_BOOLEAN,
-      Integer => TYPE_INT,
-      String => TYPE_STRING
-    }
-
-    def init_for_ruby_value(val)
-      CLASS_TO_GTYPE_MAP.each do |klass, type|
-        if val.is_a? klass
-          init type
-          return self
-        end
-      end
-      raise "Can't handle #{val.class}"
-    end
-
     def current_gtype
       @struct[:g_type]
     end
@@ -99,22 +78,19 @@ module GObject
 
     # TODO: Give more generic name
     def self.wrap_ruby_value(val)
-      new.tap { |gv| gv.set_ruby_value val }
+      new.tap { |gv| gv.__send__ :set_ruby_value, val }
     end
 
     def self.from(val)
       case val
       when self
         val
-      when nil
-        nil
       else
         wrap_ruby_value val
       end
     end
 
     def self.for_gtype(gtype)
-      return nil if gtype == TYPE_NONE
       new.tap do |it|
         it.init gtype
       end
@@ -130,6 +106,36 @@ module GObject
 
     private
 
+    def set_ruby_value(val)
+      init_for_ruby_value val if uninitialized?
+      set_value val
+    end
+
+    CLASS_TO_GTYPE_MAP = {
+      NilClass => TYPE_INVALID,
+      TrueClass => TYPE_BOOLEAN,
+      FalseClass => TYPE_BOOLEAN,
+      Integer => TYPE_INT,
+      String => TYPE_STRING
+    }
+
+    def init_for_ruby_value(val)
+      CLASS_TO_GTYPE_MAP.each do |klass, type|
+        return init type if val.is_a? klass
+      end
+      raise "Can't handle #{val.class}"
+    end
+
+    def set_none(_)
+    end
+
+    def get_none
+    end
+
+    def uninitialized?
+      current_gtype == TYPE_INVALID
+    end
+
     def set_instance_enhanced(val)
       check_type_compatibility val if val
       set_instance val

data/lib/ffi-gobject.rb

@@ -47,9 +47,11 @@ module GObject
     argument_gvalues = sig_info.arguments_to_gvalues object, args
     return_gvalue = sig_info.gvalue_for_return_value
 
-    signal_emitv argument_gvalues, signal_id, detail_quark, return_gvalue
-
-    return_gvalue
+    result = signal_emitv argument_gvalues, signal_id, detail_quark, return_gvalue
+    # NOTE: Depending on the version of GObjectIntrospection, the result will
+    # be stored in result or return_gvalue. This was changed between versions
+    # 1.44 and 1.46.
+    result || return_gvalue
   end
 
   def self.signal_connect(object, detailed_signal, data = nil, after = false, &block)

data/lib/ffi-gobject_introspection/i_base_info.rb

@@ -70,7 +70,8 @@ module GObjectIntrospection
       fetcher ||= "#{single}"
       class_eval <<-CODE
         def #{method}(name)
-          (0..(#{counter} - 1)).each do |i|
+          name = name.to_s
+          #{counter}.times do |i|
             it = #{fetcher}(i)
             return it if it.name == name
           end

data/lib/ffi-gobject_introspection/i_interface_info.rb

@@ -37,7 +37,7 @@ module GObjectIntrospection
     build_array_method :get_methods
 
     def find_method(name)
-      IFunctionInfo.wrap(Lib.g_interface_info_find_method @gobj, name)
+      IFunctionInfo.wrap(Lib.g_interface_info_find_method @gobj, name.to_s)
     end
 
     def n_signals

data/lib/ffi-gobject_introspection/i_object_info.rb

@@ -68,7 +68,7 @@ module GObjectIntrospection
     build_array_method :get_methods
 
     def find_method(name)
-      IFunctionInfo.wrap(Lib.g_object_info_find_method @gobj, name)
+      IFunctionInfo.wrap(Lib.g_object_info_find_method @gobj, name.to_s)
     end
 
     def n_signals
@@ -92,7 +92,7 @@ module GObjectIntrospection
     end
 
     def find_vfunc(name)
-      IVFuncInfo.wrap(Lib.g_object_info_find_vfunc @gobj, name)
+      IVFuncInfo.wrap(Lib.g_object_info_find_vfunc @gobj, name.to_s)
     end
     ##
     build_array_method :vfuncs

data/lib/ffi-gobject_introspection/i_struct_info.rb

@@ -24,7 +24,7 @@ module GObjectIntrospection
 
     ##
     build_array_method :get_methods
-    # There is a function g_object_info_find_method but it causes a core dump.
+    # There is a function g_struct_info_find_method but it causes a core dump.
     build_finder_method :find_method, :get_n_methods, :get_method
 
     def size

data/lib/ffi-gobject_introspection/i_union_info.rb

@@ -25,7 +25,7 @@ module GObjectIntrospection
     build_array_method :get_methods
 
     def find_method(name)
-      IFunctionInfo.wrap(Lib.g_union_info_find_method @gobj, name)
+      IFunctionInfo.wrap(Lib.g_union_info_find_method @gobj, name.to_s)
     end
 
     def size

data/lib/gir_ffi/boxed_base.rb

@@ -23,14 +23,16 @@ module GirFFI
 
     def self.copy_value_to_pointer(value, pointer, offset = 0)
       size = self::Struct.size
-      pointer.put_bytes offset, value.to_ptr.read_bytes(size), 0, size
+      bytes = if value
+                value.to_ptr.read_bytes(size)
+              else
+                "\x00" * size
+              end
+      pointer.put_bytes offset, bytes, 0, size
     end
 
-    # TODO: Make this behave more like a real .new method
-    def self._allocate
-      obj = allocate
-      obj.instance_variable_set :@struct, self::Struct.new
-      obj
+    def initialize
+      @struct = self.class::Struct.new
     end
   end
 end

data/lib/gir_ffi/builders/boxed_builder.rb

@@ -14,16 +14,6 @@ module GirFFI
         setup_constants
         stub_methods
         setup_field_accessors
-        provide_constructor
-      end
-
-      def provide_constructor
-        return if info.find_method 'new'
-
-        # TODO: Provide both new and initialize
-        (class << klass; self; end).class_eval do
-          alias_method :new, :_allocate
-        end
       end
     end
   end

data/lib/gir_ffi/builders/null_class_builder.rb

@@ -0,0 +1,13 @@
+module GirFFI
+  module Builders
+    class NullClassBuilder
+      def setup_method(_)
+        nil
+      end
+
+      def setup_instance_method(_)
+        nil
+      end
+    end
+  end
+end

data/lib/gir_ffi/builders/user_defined_builder.rb

@@ -200,9 +200,9 @@ module GirFFI
 
       def setup_constructor
         code = <<-CODE
-        def self.new
-          gptr = GObject::Lib.g_object_newv #{@gtype}, 0, nil
-          self.wrap(gptr)
+        def initialize
+          ptr = GObject::Lib.g_object_newv #{@gtype}, 0, nil
+          store_pointer(ptr)
         end
         CODE
         klass.class_eval code

data/lib/gir_ffi/class_base.rb

@@ -1,5 +1,6 @@
 require 'forwardable'
 require 'gir_ffi/registered_type_base'
+require 'gir_ffi/builders/null_class_builder'
 
 module GirFFI
   # Base class for all generated classes and structs. Contains code for dealing
@@ -8,6 +9,8 @@ module GirFFI
     extend RegisteredTypeBase
     extend Forwardable
 
+    GIR_FFI_BUILDER = Builders::NullClassBuilder.new
+
     attr_reader :struct
     def_delegators :@struct, :to_ptr
 
@@ -15,7 +18,7 @@ module GirFFI
       method_name = self.class.try_in_ancestors(:setup_instance_method, method.to_s)
 
       unless method_name
-        raise "Unable to set up instance method '#{method}' in #{self}"
+        raise NoMethodError, "undefined method `#{method}' for #{self}"
       end
 
       send method_name, *arguments, &block
@@ -27,11 +30,15 @@ module GirFFI
       other.class == self.class && to_ptr.address == other.to_ptr.address
     end
 
+    def initialize
+      raise NoMethodError
+    end
+
     def self.setup_and_call(method, arguments, &block)
       method_name = try_in_ancestors(:setup_method, method.to_s)
 
       unless method_name
-        raise "Unable to set up method '#{method}' in #{self}"
+        raise NoMethodError, "undefined method `#{method}' for #{self}"
       end
 
       send method_name, *arguments, &block
@@ -44,6 +51,7 @@ module GirFFI
           return result if result
         end
       end
+      nil
     end
 
     def self.to_ffi_type
@@ -58,10 +66,6 @@ module GirFFI
       gir_ffi_builder.setup_instance_method name
     end
 
-    class << self
-      undef new
-    end
-
     # Wrap the passed pointer in an instance of the current class, or a
     # descendant type if applicable.
     def self.wrap(ptr)

data/lib/gir_ffi/in_out_pointer.rb

@@ -52,7 +52,7 @@ module GirFFI
     end
 
     def clear
-      set_value nil_value
+      put_bytes 0, "\x00" * value_type_size, 0, value_type_size
     end
 
     def self.for(type)
@@ -72,9 +72,5 @@ module GirFFI
     def value_type_size
       @value_type_size ||= FFI.type_size value_ffi_type
     end
-
-    def nil_value
-      value_ffi_type == :pointer ? nil : 0
-    end
   end
 end

data/lib/gir_ffi/info_ext/i_registered_type_info.rb

@@ -16,6 +16,10 @@ module GirFFI
         info = find_method method
         return info if info && info.method?
       end
+
+      def find_method(method)
+        raise 'Must be overridden in subclass'
+      end
     end
   end
 end

data/lib/gir_ffi/user_defined_type_info.rb

@@ -27,6 +27,10 @@ module GirFFI
       @vfunc_implementations << VFuncImplementation.new(name, implementation)
     end
 
+    def find_method(_method)
+      nil
+    end
+
     def find_instance_method(_method)
       nil
     end

data/lib/gir_ffi/version.rb

@@ -1,4 +1,4 @@
 # Current GirFFI version
 module GirFFI
-  VERSION = '0.8.1'
+  VERSION = '0.8.2'
 end

data/test/ffi-glib/ruby_closure_test.rb

@@ -38,7 +38,7 @@ describe GObject::RubyClosure do
 
     it "stores the closure's return value in the proper gvalue" do
       c = GObject::RubyClosure.new { 2 }
-      gv = GObject::Value.new
+      gv = GObject::Value.new.init GObject::TYPE_INT
       GObject::RubyClosure.marshaller(c, gv, nil, nil, nil)
       assert_equal 2, gv.get_value
     end

data/test/ffi-gobject/value_test.rb

@@ -18,6 +18,23 @@ describe GObject::Value do
     end
   end
 
+  describe '.for_gtype' do
+    it 'handles char' do
+      gv = GObject::Value.for_gtype GObject::TYPE_CHAR
+      gv.current_gtype.must_equal GObject::TYPE_CHAR
+    end
+
+    it 'handles invalid type' do
+      gv = GObject::Value.for_gtype GObject::TYPE_INVALID
+      gv.current_gtype.must_equal GObject::TYPE_INVALID
+    end
+
+    it 'handles void type' do
+      gv = GObject::Value.for_gtype GObject::TYPE_NONE
+      gv.current_gtype.must_equal GObject::TYPE_INVALID
+    end
+  end
+
   describe '::wrap_ruby_value' do
     it 'wraps a boolean false' do
       gv = GObject::Value.wrap_ruby_value false
@@ -40,6 +57,12 @@ describe GObject::Value do
       gv = GObject::Value.wrap_ruby_value 'Some Random String'
       assert_equal 'Some Random String', gv.get_string
     end
+
+    it 'wraps nil' do
+      gv = GObject::Value.wrap_ruby_value nil
+      assert_instance_of GObject::Value, gv
+      assert_equal nil, gv.get_value
+    end
   end
 
   describe '#set_value' do
@@ -247,6 +270,12 @@ describe GObject::Value do
       gv2.current_gtype_name.must_equal 'gint'
       gv2.get_value.must_equal 21
     end
+
+    it 'creates a null GValue from a Ruby nil' do
+      gv = GObject::Value.from nil
+      gv.current_gtype.must_equal GObject::TYPE_INVALID
+      gv.get_value.must_equal nil
+    end
   end
 
   describe '#set_value' do

data/test/ffi-gobject_introspection/i_interface_info_test.rb

@@ -0,0 +1,21 @@
+require 'introspection_test_helper'
+
+describe GObjectIntrospection::IInterfaceInfo do
+  let(:object_info) { get_introspection_data('GObject', 'TypePlugin') }
+
+  describe '#find_method' do
+    it 'finds a method by name string' do
+      object_info.find_method('complete_interface_info').wont_be_nil
+    end
+
+    it 'finds a method by name symbol' do
+      object_info.find_method(:complete_interface_info).wont_be_nil
+    end
+  end
+
+  describe '#type_name' do
+    it 'returns the correct name' do
+      object_info.type_name.must_equal 'GTypePlugin'
+    end
+  end
+end

data/test/ffi-gobject_introspection/i_object_info_test.rb

@@ -4,9 +4,23 @@ describe GObjectIntrospection::IObjectInfo do
   let(:object_info) { get_introspection_data('GObject', 'Object') }
 
   describe '#find_vfunc' do
-    it 'finds a vfunc by name' do
+    it 'finds a vfunc by name string' do
       object_info.find_vfunc('finalize').wont_be_nil
     end
+
+    it 'finds a vfunc by name symbol' do
+      object_info.find_vfunc(:finalize).wont_be_nil
+    end
+  end
+
+  describe '#find_method' do
+    it 'finds a method by name string' do
+      object_info.find_method('bind_property').wont_be_nil
+    end
+
+    it 'finds a method by name symbol' do
+      object_info.find_method(:bind_property).wont_be_nil
+    end
   end
 
   describe '#type_name' do

data/test/ffi-gobject_introspection/i_struct_info_test.rb

@@ -0,0 +1,21 @@
+require 'introspection_test_helper'
+
+describe GObjectIntrospection::IStructInfo do
+  let(:object_info) { get_introspection_data('GObject', 'Closure') }
+
+  describe '#find_method' do
+    it 'finds a method by name string' do
+      object_info.find_method('new_simple').wont_be_nil
+    end
+
+    it 'finds a method by name symbol' do
+      object_info.find_method(:new_simple).wont_be_nil
+    end
+  end
+
+  describe '#type_name' do
+    it 'returns the correct name' do
+      object_info.type_name.must_equal 'GClosure'
+    end
+  end
+end

data/test/ffi-gobject_introspection/i_union_info_test.rb

@@ -0,0 +1,15 @@
+require 'introspection_test_helper'
+
+describe GObjectIntrospection::IUnionInfo do
+  let(:object_info) { get_introspection_data('GLib', 'Mutex') }
+
+  describe '#find_method' do
+    it 'finds a method by name string' do
+      object_info.find_method('clear').wont_be_nil
+    end
+
+    it 'finds a method by name symbol' do
+      object_info.find_method(:clear).wont_be_nil
+    end
+  end
+end

data/test/gir_ffi/class_base_test.rb

@@ -92,8 +92,7 @@ describe GirFFI::ClassBase do
           'correct-result'
         end
 
-        def self.new
-          allocate
+        def initialize
         end
       end
       klass.const_set :GIR_FFI_BUILDER, builder
@@ -101,6 +100,19 @@ describe GirFFI::ClassBase do
       result = klass.setup_and_call :foo, []
       result.must_equal 'correct-result'
     end
+
+    it 'raises a sensible error if the method is not found' do
+      expect(builder = Object.new).to receive(:setup_method).with('foo').and_return nil
+      klass = Class.new GirFFI::ClassBase do
+        def initialize
+        end
+      end
+      klass.const_set :GIR_FFI_BUILDER, builder
+
+      proc { klass.setup_and_call :foo, [] }.
+        must_raise(NoMethodError).message.
+        must_match(/^undefined method `foo' for/)
+    end
   end
 
   describe '#setup_and_call' do
@@ -114,8 +126,7 @@ describe GirFFI::ClassBase do
         def foo
         end
 
-        def self.new
-          allocate
+        def initialize
         end
       end
       sub_klass.const_set :GIR_FFI_BUILDER, sub_builder
@@ -132,8 +143,7 @@ describe GirFFI::ClassBase do
           'correct-result'
         end
 
-        def self.new
-          allocate
+        def initialize
         end
       end
       klass.const_set :GIR_FFI_BUILDER, builder
@@ -143,5 +153,20 @@ describe GirFFI::ClassBase do
       result = obj.setup_and_call :foo, []
       result.must_equal 'correct-result'
     end
+
+    it 'raises a sensible error if the method is not found' do
+      expect(builder = Object.new).to receive(:setup_instance_method).with('foo').and_return nil
+      klass = Class.new GirFFI::ClassBase do
+        def initialize
+        end
+      end
+      klass.const_set :GIR_FFI_BUILDER, builder
+
+      obj = klass.new
+
+      proc { obj.setup_and_call :foo, [] }.
+        must_raise(NoMethodError).message.
+        must_match(/^undefined method `foo' for/)
+    end
   end
 end

data/test/gir_ffi/in_out_pointer_test.rb

@@ -57,11 +57,30 @@ describe GirFFI::InOutPointer do
 
   describe '.for' do
     it 'handles :gboolean' do
-      GirFFI::InOutPointer.for :gboolean
+      result = GirFFI::InOutPointer.for :gboolean
+      result.to_value.must_equal false
     end
 
     it 'handles :utf8' do
-      GirFFI::InOutPointer.for :utf8
+      result = GirFFI::InOutPointer.for :utf8
+      result.to_value.must_be :null?
+    end
+
+    it 'handles GObject::Value' do
+      result = GirFFI::InOutPointer.for GObject::Value
+      type_size = FFI.type_size GObject::Value
+      expected = "\x00" * type_size
+      result.to_value.read_bytes(type_size).must_equal expected
+    end
+  end
+
+  describe '#set_value' do
+    it 'works setting the value to nil for GObject::Value' do
+      pointer = GirFFI::InOutPointer.from GObject::Value, GObject::Value.from(3)
+      pointer.set_value nil
+      type_size = FFI.type_size GObject::Value
+      expected = "\x00" * type_size
+      pointer.to_value.read_bytes(type_size).must_equal expected
     end
   end
 

data/test/gir_ffi/user_defined_type_info_test.rb

@@ -76,4 +76,13 @@ describe GirFFI::UserDefinedTypeInfo do
       info.g_name.must_equal 'bar'
     end
   end
+
+  describe '#find_method' do
+    let(:klass) { Object.new }
+    let(:info) { GirFFI::UserDefinedTypeInfo.new klass }
+
+    it 'finds no methods' do
+      info.find_method('foo').must_equal nil
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gir_ffi
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.8.2
 platform: ruby
 authors:
 - Matijs van Zuijlen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-04 00:00:00.000000000 Z
+date: 2015-10-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -189,6 +189,7 @@ files:
 - lib/gir_ffi/builders/method_template.rb
 - lib/gir_ffi/builders/module_builder.rb
 - lib/gir_ffi/builders/null_argument_builder.rb
+- lib/gir_ffi/builders/null_class_builder.rb
 - lib/gir_ffi/builders/null_convertor.rb
 - lib/gir_ffi/builders/null_return_value_builder.rb
 - lib/gir_ffi/builders/object_builder.rb
@@ -278,10 +279,13 @@ files:
 - test/ffi-gobject_introspection/i_constant_info_test.rb
 - test/ffi-gobject_introspection/i_enum_info_test.rb
 - test/ffi-gobject_introspection/i_function_info_test.rb
+- test/ffi-gobject_introspection/i_interface_info_test.rb
 - test/ffi-gobject_introspection/i_object_info_test.rb
 - test/ffi-gobject_introspection/i_registered_type_info_test.rb
 - test/ffi-gobject_introspection/i_repository_test.rb
+- test/ffi-gobject_introspection/i_struct_info_test.rb
 - test/ffi-gobject_introspection/i_type_info_test.rb
+- test/ffi-gobject_introspection/i_union_info_test.rb
 - test/ffi-gobject_introspection/lib_test.rb
 - test/ffi-gobject_test.rb
 - test/gir_ffi-base/glib/boolean_test.rb