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 88726 - [PATCH] Examples
[PATCH] Examples
Status: RESOLVED FIXED
Product: gnomemm
Classification: Deprecated
Component: libgnomecanvasmm
2.0
Other All
: Normal normal
: ---
Assigned To: gtkmm-forge
gtkmm-forge
Depends on:
Blocks:
 
 
Reported: 2002-07-21 15:01 UTC by Erwin J. van Eijk
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to a cleaner example. (5.62 KB, patch)
2002-07-21 15:02 UTC, Erwin J. van Eijk
none Details | Review
New patch - no more using and renamed the signal handlers. (5.57 KB, patch)
2002-07-23 18:28 UTC, Erwin J. van Eijk
none Details | Review

Description Erwin J. van Eijk 2002-07-21 15:01:24 UTC
One of the examples, canvas_events caused an infinite recursion to occur.
I've patched canvas_events to be more like canvas.cc, and not to cause the
recursion.
Comment 1 Erwin J. van Eijk 2002-07-21 15:02:25 UTC
Created attachment 9991 [details] [review]
Patch to a cleaner example.
Comment 2 Murray Cumming 2002-07-21 19:31:13 UTC
Thanks, but:

I prefer examples not to use "using", and I prefer signal handlers to
start with "on_". I'm not actually the libgnomecanvasmm maintainer though.
Comment 3 Erwin J. van Eijk 2002-07-23 18:26:27 UTC
Ok, that's no problem. I've updated the patch more to your liking.
Comment 4 Erwin J. van Eijk 2002-07-23 18:28:02 UTC
Created attachment 10023 [details] [review]
New patch - no more using and renamed the signal handlers.
Comment 5 Murray Cumming 2002-07-24 07:29:41 UTC
Applied. Thanks. Please remember to patch the ChangeLog in future.