Make Full the default transfer for returned foreign structs.

For that we also need to read the annotations for foreign structs
before guessing the transfer mode of return values.

https://bugzilla.gnome.org/show_bug.cgi?id=629188

3 files changed, 15 insertions, 0 deletions

diff --git a/tests/scanner/Foo-1.0-expected.gir b/tests/scanner/Foo-1.0-expected.gir
index a0cb696d..378852ef 100644
--- a/tests/scanner/Foo-1.0-expected.gir
+++ b/tests/scanner/Foo-1.0-expected.gir
@@ -260,6 +260,11 @@ and/or use gtk-doc annotations.  -->
           <type name="ForeignStruct" c:type="FooForeignStruct*"/>
         </return-value>
       </constructor>
+      <method name="copy" c:identifier="foo_foreign_struct_copy">
+        <return-value transfer-ownership="full">
+          <type name="ForeignStruct" c:type="FooForeignStruct*"/>
+        </return-value>
+      </method>
     </record>
     <interface name="Interface"
                c:symbol-prefix="interface"
diff --git a/tests/scanner/foo.c b/tests/scanner/foo.c
index a2901f1b..8b8b40d0 100644
--- a/tests/scanner/foo.c
+++ b/tests/scanner/foo.c
@@ -717,6 +717,15 @@ foo_foreign_struct_new (void)
   return g_slice_new0 (FooForeignStruct);
 }
 
+FooForeignStruct*
+foo_foreign_struct_copy (FooForeignStruct *original)
+{
+    FooForeignStruct *copy;
+    copy = foo_foreign_struct_new ();
+    copy->foo = original->foo;
+    return copy;
+}
+
 /**
  * foo_test_varargs_callback: (skip)
  *
diff --git a/tests/scanner/foo.h b/tests/scanner/foo.h
index 6c572666..e648e609 100644
--- a/tests/scanner/foo.h
+++ b/tests/scanner/foo.h
@@ -424,6 +424,7 @@ struct _FooForeignStruct
 };
 
 FooForeignStruct* foo_foreign_struct_new (void);
+FooForeignStruct* foo_foreign_struct_copy (FooForeignStruct *original);
 
 /* This one should be a global, not a method on UtilityObject since
  * it's a separate namespace.