]> git.ipfire.org Git - thirdparty/vala.git/commitdiff
gsignal: Use guint to represent flags-enum in custom GMarshalFunc
authorRico Tzschichholz <ricotz@ubuntu.com>
Sun, 10 Mar 2019 17:14:09 +0000 (18:14 +0100)
committerRico Tzschichholz <ricotz@ubuntu.com>
Wed, 10 Apr 2019 14:09:59 +0000 (16:09 +0200)
codegen/valagsignalmodule.vala
tests/Makefile.am
tests/objects/signals-enum-marshal.vala [new file with mode: 0644]

index fd1a1129d8c1d794b0c6d18bdad2ea33eeb1aa6c..e3a12f58faa5383ce7bfe16ad86505f18de20a75 100644 (file)
@@ -68,7 +68,12 @@ public class Vala.GSignalModule : GObjectModule {
                                return "gpointer";
                        }
                } else if (t.data_type is Enum) {
-                       return "gint";
+                       var en = (Enum) t.data_type;
+                       if (en.is_flags) {
+                               return "guint";
+                       } else {
+                               return "gint";
+                       }
                } else if (t is ArrayType) {
                        return "gpointer";
                } else if (t is ErrorType) {
index caea87f06abe4040d4b6ebbf4794452bee633159..9de37647440744f3928cd9d3dc3cc6c779e6041f 100644 (file)
@@ -277,6 +277,7 @@ TESTS = \
        objects/property-static.vala \
        objects/regex.vala \
        objects/signals.vala \
+       objects/signals-enum-marshal.vala \
        objects/signals-delegate.vala \
        objects/signals-lambda-delegate.vala \
        objects/test-025.vala \
diff --git a/tests/objects/signals-enum-marshal.vala b/tests/objects/signals-enum-marshal.vala
new file mode 100644 (file)
index 0000000..412fa18
--- /dev/null
@@ -0,0 +1,21 @@
+enum Foo {
+       FOO
+}
+
+[Flags]
+enum Bar {
+       BAR
+}
+
+class Manam : Object {
+       public signal void foo (Foo f, string s);
+       public signal void bar (Bar b, string s);
+}
+
+void main () {
+       var manam = new Manam ();
+       manam.foo.connect (() => {});
+       manam.foo (Foo.FOO, "foo");
+       manam.bar.connect (() => {});
+       manam.bar (Bar.BAR, "bar");
+}