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 636336 - Gtk: add an override for Gtk.main_quit
Gtk: add an override for Gtk.main_quit
Status: RESOLVED FIXED
Product: pygobject
Classification: Bindings
Component: introspection
unspecified
Other All
: Normal normal
: ---
Assigned To: Nobody's working on this now (help wanted and appreciated)
Python bindings maintainers
Depends on:
Blocks:
 
 
Reported: 2010-12-03 01:49 UTC by Johan (not receiving bugmail) Dahlin
Modified: 2010-12-04 00:56 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Gtk: add an override for Gtk.main_quit (976 bytes, patch)
2010-12-03 01:49 UTC, Johan (not receiving bugmail) Dahlin
none Details | Review
Gtk: add an override for Gtk.main_quit (2.23 KB, patch)
2010-12-03 12:20 UTC, Johan (not receiving bugmail) Dahlin
committed Details | Review

Description Johan (not receiving bugmail) Dahlin 2010-12-03 01:49:10 UTC
Override gtk.main_quit so that it can be used as a callback
to signals, most importantly to the destroy signal of a widget:

   widget.connect('destroy', gtk.main_quit)

This is compatible with what PyGTK does.
Comment 1 Johan (not receiving bugmail) Dahlin 2010-12-03 01:49:13 UTC
Created attachment 175752 [details] [review]
Gtk: add an override for Gtk.main_quit
Comment 2 Johan (not receiving bugmail) Dahlin 2010-12-03 12:20:26 UTC
Created attachment 175776 [details] [review]
Gtk: add an override for Gtk.main_quit

Override gtk.main_quit so that it can be used as a callback
to signals, most importantly to the destroy signal of a widget:

   widget.connect('destroy', gtk.main_quit)

This is compatible with what PyGTK does.

Updated version with a little bit of generic infrastructure.
Comment 3 johnp 2010-12-03 18:09:22 UTC
Comment on attachment 175776 [details] [review]
Gtk: add an override for Gtk.main_quit

looks good.  I'll commit it today