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 565670 - recycling of gtk.CellRendererText()
recycling of gtk.CellRendererText()
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.25.x
Other All
: Normal minor
: 2.26.0
Assigned To: Orca Maintainers
Orca Maintainers
Depends on:
Blocks: 491756
 
 
Reported: 2008-12-26 06:38 UTC by Mesar Hameed
Modified: 2009-03-10 14:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
replacing instances by ref (5.30 KB, patch)
2008-12-26 06:46 UTC, Mesar Hameed
none Details | Review
Pylinted patch (5.91 KB, patch)
2009-01-05 14:17 UTC, Willie Walker
committed Details | Review

Description Mesar Hameed 2008-12-26 06:38:53 UTC
Please describe the problem:
Happy Christmas :)

According to:
http://www.moeraki.com/pygtkreference/pygtk2reference/class-gtkcellrenderer.html
just before the "methods" (heading at level 2)

"The primary use of a gtk.CellRenderer is for drawing a certain graphical elements on a gtk.gdk.Drawable. Typically, one cell renderer is used to draw many cells on the screen. To this extent, it isn't expected that a CellRenderer keep any permanent state around. Instead, any state is set just prior to use using the GObjects property system. Then, the cell is measured using the get_size() method. Finally, the cell is rendered in the correct location using the render() method."

replacing 10 instances by a reference.
patch to be attached.

Thank you


Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Mesar Hameed 2008-12-26 06:46:37 UTC
Created attachment 125326 [details] [review]
replacing instances by ref

promised patch.
Comment 2 Willie Walker 2009-01-05 14:17:56 UTC
Created attachment 125784 [details] [review]
Pylinted patch

Here's a pylinted version of the patch.  Many thanks for providing the original patch!  I'm going to put this in just for trunk and not gnome-2-24 -- it seems like it provides a performance improvement for the preferences GUI only, so I don't believe the overall user benefit is great enough for GNOME 2.24.  If you disagree, though, please let me know and I can check it in to gnome-2-24 after it's soaked this week.