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 632413 - GdkCairo docs
GdkCairo docs
Status: RESOLVED FIXED
Product: gnome-perl
Classification: Bindings
Component: Gtk2
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: gtk2-perl-bugs
gtk2-perl-bugs
Depends on:
Blocks:
 
 
Reported: 2010-10-17 23:28 UTC by Kevin Ryde
Modified: 2011-09-30 21:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
doc patch (3.76 KB, patch)
2010-10-17 23:28 UTC, Kevin Ryde
committed Details | Review
4-arg rectangle docs (1.28 KB, patch)
2011-08-30 23:46 UTC, Kevin Ryde
committed Details | Review

Description Kevin Ryde 2010-10-17 23:28:29 UTC
Created attachment 172572 [details] [review]
doc patch

Are the Gtk2::Gdk::Cairo::Context methods meant to be used with all kinds of Cairo::Context objects too?  Is that in fact the usual way to draw from a pixbuf or pixmap onto a printer (Gtk2::PrintOperation stuff) etc?  Perhaps a bit for the docs as attached.

The args listed for ->rectangle come out in reverse order with current GenPod.pm, but I think that might be down to it rather than the pod.
Comment 1 Torsten Schoenfeld 2011-08-16 19:47:46 UTC
Yes, this is indeed how these xsubs can also be used.  The second way of
calling ->rectangle(), however, is just for compatibility with
Cairo::Context::rectangle(), and I thus prefer it not appearing in the docs.
With this tweak made, I commited the patch.  Thanks!
Comment 2 Kevin Ryde 2011-08-30 23:46:24 UTC
Created attachment 195264 [details] [review]
4-arg rectangle docs

I think it'd be good to show the x,y,width,height too, so it's clear that can still be used on the gdk subclass.

(It all gets pretty confusing, and the fact the superclass calling style still works in the subclass isn't supposed to be a secret is it :-)
Comment 3 Torsten Schoenfeld 2011-09-30 21:11:37 UTC
Alright, convinced.