After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 663289 - Some signals expect incorrect number of arguments
Some signals expect incorrect number of arguments
Status: RESOLVED FIXED
Product: seed
Classification: Bindings
Component: libseed
git master
Other All
: Normal normal
: ---
Assigned To: seed-maint
Depends on:
Blocks:
 
 
Reported: 2011-11-03 06:49 UTC by Sunil Mohan Adapa
Modified: 2011-11-03 07:19 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Add test case that fails because of the bug (1.87 KB, patch)
2011-11-03 07:02 UTC, Sunil Mohan Adapa 		none Details | Review
Patch to fix the bug (745 bytes, patch)
2011-11-03 07:18 UTC, Sunil Mohan Adapa 		none Details | Review

Description Sunil Mohan Adapa 2011-11-03 06:49:10 UTC 
Sometimes signals expect more arguments than specified. The problem can be reproduced in the following manner:

Declare two signals. The first signal should have non-zero number of arguments (say two). The second signal should not have 'parameters' value set. Upon trying to emit the second signal, the following error message is shown:

ArgumentError Signal: less-args for type TestObject expected 2 arguments, got 0
Comment 1 Alan Knowles 2011-11-03 07:00:38 UTC 
Can you provide a code example showing this.

Thanks
Alan
Comment 2 Sunil Mohan Adapa 2011-11-03 07:02:18 UTC 
Created attachment 200571 [details] [review]
Add test case that fails because of the bug
Comment 3 Sunil Mohan Adapa 2011-11-03 07:18:07 UTC 
Created attachment 200572 [details] [review]
Patch to fix the bug

The attached patch fixed the problem for me
Comment 4 Alan Knowles 2011-11-03 07:19:00 UTC 
Yes, I also reset the param_types just to be sure.

It's committed now.

Thanks
Alan