]> 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 12:48:40 +0000 (14:48 +0200)
codegen/valagsignalmodule.vala
tests/Makefile.am
tests/objects/signals-enum-marshal.vala [new file with mode: 0644]

index c60e16cab0c31b7e57d944f1f3cbcbe6c1ac7d98..5dbf08873c6d503bfc7436e9c02869ee6c69fb55 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 51783c72384f7204d35547f4f2937c1a8249dd38..576e667f363387e59cf2be2b232c92323f3cd76a 100644 (file)
@@ -233,6 +233,7 @@ TESTS = \
        objects/property-static.vala \
        objects/regex.vala \
        objects/signals.vala \
+       objects/signals-enum-marshal.vala \
        objects/test-025.vala \
        objects/test-026.vala \
        objects/test-029.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");
+}