gir_ffi 0.7.10 → 0.8.0

data/test/ffi-gobject_introspection/i_registered_type_info_test.rb

@@ -3,9 +3,9 @@ require 'introspection_test_helper'
 describe GObjectIntrospection::IRegisteredTypeInfo do
   describe '#get_type_name' do
     describe 'for an interface' do
-      let(:registered_type_info) {
+      let(:registered_type_info) do
         get_introspection_data('GIMarshallingTests', 'Interface')
-      }
+      end
 
       it 'returns interface name' do
         registered_type_info.type_name.must_equal 'GIMarshallingTestsInterface'
@@ -13,9 +13,9 @@ describe GObjectIntrospection::IRegisteredTypeInfo do
     end
 
     describe 'for a type that is not an interface' do
-      let(:registered_type_info) {
+      let(:registered_type_info) do
         get_introspection_data('GIMarshallingTests', 'Enum')
-      }
+      end
 
       it 'returns nil' do
         registered_type_info.type_name.must_be_nil

data/test/ffi-gobject_introspection/i_type_info_test.rb

@@ -9,9 +9,9 @@ describe GObjectIntrospection::ITypeInfo do
 
     it 'raises an error' do
       skip unless vfunc_info
-      proc {
+      proc do
         type_info.name
-      }.must_raise RuntimeError
+      end.must_raise RuntimeError
     end
   end
 

data/test/ffi-gobject_test.rb

@@ -122,10 +122,10 @@ describe GObject do
         sb = Regress::TestSimpleBoxedA.new
         sb.some_int = 23
 
-        GObject.signal_connect(o, 'test-with-static-scope-arg', 2) { |_instance, object, user_data|
+        GObject.signal_connect(o, 'test-with-static-scope-arg', 2) do |_instance, object, user_data|
           @a = user_data
           @b = object
-        }
+        end
         GObject.signal_emit o, 'test-with-static-scope-arg', sb
       end
 

data/test/gir_ffi/builders/argument_builder_test.rb

@@ -6,9 +6,9 @@ describe GirFFI::Builders::ArgumentBuilder do
 
   describe 'for an argument with direction :in' do
     describe 'for :callback' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('Regress', 'test_callback_destroy_notify').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = Regress::TestCallbackUserData.from(callback)']
@@ -20,10 +20,10 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :zero_terminated' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests',
                                'array_in_len_zero_terminated').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::ZeroTerminated.from(:gint32, ints)']
@@ -81,9 +81,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :flags' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'flags_out').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for GIMarshallingTests::Flags']
@@ -95,9 +95,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :object' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_method_introspection_data('Regress', 'TestObj', 'null_out').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, Regress::TestObj]']
@@ -110,9 +110,9 @@ describe GirFFI::Builders::ArgumentBuilder do
 
     describe 'for :struct' do
       describe 'when not allocated by the caller' do
-        let(:arg_info) {
+        let(:arg_info) do
           get_introspection_data('GIMarshallingTests', 'boxed_struct_out').args[0]
-        }
+        end
 
         it 'has the correct value for #pre_conversion' do
           builder.pre_conversion.must_equal [
@@ -126,9 +126,9 @@ describe GirFFI::Builders::ArgumentBuilder do
       end
 
       describe 'when allocated by the caller' do
-        let(:arg_info) {
+        let(:arg_info) do
           get_method_introspection_data('Regress', 'TestStructA', 'clone').args[0]
-        }
+        end
 
         it 'has the correct value for #pre_conversion' do
           builder.pre_conversion.must_equal ['_v1 = Regress::TestStructA.new']
@@ -141,9 +141,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :strv' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'gstrv_out').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, :strv]']
@@ -156,9 +156,9 @@ describe GirFFI::Builders::ArgumentBuilder do
 
     describe 'for :array' do
       describe 'when allocated by the callee' do
-        let(:arg_info) {
+        let(:arg_info) do
           get_introspection_data('GIMarshallingTests', 'garray_utf8_none_out').args[0]
-        }
+        end
 
         it 'has the correct value for #pre_conversion' do
           builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, :array]']
@@ -170,9 +170,9 @@ describe GirFFI::Builders::ArgumentBuilder do
       end
 
       describe 'when allocated by the caller' do
-        let(:function_info) {
+        let(:function_info) do
           get_introspection_data('GIMarshallingTests', 'garray_utf8_full_out_caller_allocated')
-        }
+        end
 
         let(:arg_info) { function_info.args[0] }
 
@@ -191,9 +191,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :ptr_array' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'gptrarray_utf8_none_out').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, :ptr_array]']
@@ -205,9 +205,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :error' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'gerror_out').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, :error]']
@@ -220,9 +220,9 @@ describe GirFFI::Builders::ArgumentBuilder do
 
     describe 'for :c' do
       describe 'with fixed size' do
-        let(:arg_info) {
+        let(:arg_info) do
           get_introspection_data('GIMarshallingTests', 'array_fixed_out').args[0]
-        }
+        end
 
         it 'has the correct value for #pre_conversion' do
           builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, :c]']
@@ -234,9 +234,9 @@ describe GirFFI::Builders::ArgumentBuilder do
       end
 
       describe 'with separate size parameter' do
-        let(:arg_info) {
+        let(:arg_info) do
           get_introspection_data('GIMarshallingTests', 'array_out').args[0]
-        }
+        end
 
         let(:length_argument) { Object.new }
         before do
@@ -255,9 +255,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :glist' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'glist_utf8_none_out').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, :glist]']
@@ -269,9 +269,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :gslist' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'gslist_utf8_none_out').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, :gslist]']
@@ -283,9 +283,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :ghash' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'ghashtable_utf8_none_out').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, [:ghash, :utf8, :utf8]]']
@@ -299,9 +299,9 @@ describe GirFFI::Builders::ArgumentBuilder do
 
   describe 'for an argument with direction :inout' do
     describe 'for :enum' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'enum_inout').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for GIMarshallingTests::Enum',
@@ -314,9 +314,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :flags' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'no_type_flags_inout').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for GIMarshallingTests::NoTypeFlags',
@@ -329,9 +329,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :gint32' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'int32_inout_min_max').args[0]
-      }
+      end
 
       it 'has the correct value for method_argument_name' do
         builder.method_argument_name.must_equal "#{arg_info.name}"
@@ -348,14 +348,14 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for an array length' do
-      let(:function_info) {
+      let(:function_info) do
         get_introspection_data('Regress', 'test_array_int_inout')
-      }
+      end
       let(:arg_info) { function_info.args[0] }
       let(:array_arg_info) { function_info.args[1] }
-      let(:array_arg_builder) {
+      let(:array_arg_builder) do
         GirFFI::Builders::ArgumentBuilder.new(var_gen, array_arg_info)
-      }
+      end
 
       before do
         builder.array_arg = array_arg_builder
@@ -373,9 +373,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :strv' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'gstrv_inout').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, :strv]',
@@ -388,9 +388,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :ptr_array' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'gptrarray_utf8_none_inout').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for [:pointer, :ptr_array]',
@@ -403,9 +403,9 @@ describe GirFFI::Builders::ArgumentBuilder do
     end
 
     describe 'for :utf8' do
-      let(:arg_info) {
+      let(:arg_info) do
         get_introspection_data('GIMarshallingTests', 'utf8_none_inout').args[0]
-      }
+      end
 
       it 'has the correct value for #pre_conversion' do
         builder.pre_conversion.must_equal ['_v1 = GirFFI::InOutPointer.for :utf8',
@@ -419,9 +419,9 @@ describe GirFFI::Builders::ArgumentBuilder do
 
     describe 'for :c' do
       describe 'with fixed size' do
-        let(:arg_info) {
+        let(:arg_info) do
           get_introspection_data('GIMarshallingTests', 'array_fixed_inout').args[0]
-        }
+        end
 
         it 'has the correct value for #pre_conversion' do
           builder.pre_conversion.must_equal [
@@ -437,14 +437,14 @@ describe GirFFI::Builders::ArgumentBuilder do
       end
 
       describe 'with separate size parameter' do
-        let(:function_info) {
+        let(:function_info) do
           get_introspection_data('Regress', 'test_array_int_inout')
-        }
+        end
         let(:length_arg_info) { function_info.args[0] }
         let(:arg_info) { function_info.args[1] }
-        let(:length_arg_builder) {
+        let(:length_arg_builder) do
           GirFFI::Builders::ArgumentBuilder.new(var_gen, length_arg_info)
-        }
+        end
 
         before do
           builder.length_arg = length_arg_builder
@@ -465,9 +465,9 @@ describe GirFFI::Builders::ArgumentBuilder do
   end
 
   describe 'for a skipped argument with direction :in' do
-    let(:arg_info) {
+    let(:arg_info) do
       get_method_introspection_data('Regress', 'TestObj', 'skip_param').args[2]
-    }
+    end
 
     it 'has the correct value for method_argument_name' do
       builder.method_argument_name.must_be_nil
@@ -483,9 +483,9 @@ describe GirFFI::Builders::ArgumentBuilder do
   end
 
   describe 'for a skipped argument with direction :inout' do
-    let(:arg_info) {
+    let(:arg_info) do
       get_method_introspection_data('Regress', 'TestObj', 'skip_inout_param').args[3]
-    }
+    end
 
     it 'has the correct value for method_argument_name' do
       builder.method_argument_name.must_be_nil
@@ -502,9 +502,9 @@ describe GirFFI::Builders::ArgumentBuilder do
   end
 
   describe 'for a skipped argument with direction :out' do
-    let(:arg_info) {
+    let(:arg_info) do
       get_method_introspection_data('Regress', 'TestObj', 'skip_out_param').args[1]
-    }
+    end
 
     it 'has the correct value for method_argument_name' do
       builder.method_argument_name.must_be_nil

data/test/gir_ffi/builders/callback_argument_builder_test.rb

@@ -6,10 +6,10 @@ describe GirFFI::Builders::CallbackArgumentBuilder do
 
   describe 'for an argument with direction :out' do
     describe 'for :zero_terminated' do
-      let(:vfunc_info) {
+      let(:vfunc_info) do
         get_vfunc_introspection_data('GIMarshallingTests', 'Object',
                                      'vfunc_array_out_parameter')
-      }
+      end
       let(:arg_info) { vfunc_info.args[0] }
 
       before { skip unless vfunc_info }
@@ -44,18 +44,18 @@ describe GirFFI::Builders::CallbackArgumentBuilder do
   end
 
   describe 'for an argument with direction :inout' do
-    let(:callback_info) {
+    let(:callback_info) do
       get_introspection_data('Regress',
                              'TestCallbackArrayInOut')
-    }
+    end
     let(:array_arg_info) { callback_info.args[0] }
-    let(:array_arg_builder) {
+    let(:array_arg_builder) do
       GirFFI::Builders::CallbackArgumentBuilder.new(var_gen, array_arg_info)
-    }
+    end
     let(:length_arg_info) { callback_info.args[1] }
-    let(:length_arg_builder) {
+    let(:length_arg_builder) do
       GirFFI::Builders::CallbackArgumentBuilder.new(var_gen, length_arg_info)
-    }
+    end
 
     before do
       skip unless callback_info

data/test/gir_ffi/builders/callback_builder_test.rb

@@ -50,10 +50,10 @@ describe GirFFI::Builders::CallbackBuilder do
     end
 
     describe 'for a callback with one out argument' do
-      let(:callback_info) {
+      let(:callback_info) do
         get_introspection_data('GIMarshallingTests',
                                'CallbackOneOutParameter')
-      }
+      end
       it 'returns a valid mapping method' do
         expected = <<-CODE.reset_indentation
         def self.call_with_argument_mapping(_proc, a)
@@ -68,10 +68,10 @@ describe GirFFI::Builders::CallbackBuilder do
     end
 
     describe 'for a callback with an inout array argument' do
-      let(:callback_info) {
+      let(:callback_info) do
         get_introspection_data('Regress',
                                'TestCallbackArrayInOut')
-      }
+      end
       it 'returns a valid mapping method' do
         skip unless callback_info
         expected = <<-CODE.reset_indentation