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 604403 - Gtk2::Adjustment new 2.14 funcs
Gtk2::Adjustment new 2.14 funcs
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: 2009-12-11 22:10 UTC by Kevin Ryde
Modified: 2010-04-20 21:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
adding new 2.14 funcs (3.93 KB, patch)
2009-12-11 22:10 UTC, Kevin Ryde
committed Details | Review

Description Kevin Ryde 2009-12-11 22:10:03 UTC
Created attachment 149613 [details] [review]
adding new 2.14 funcs

Gtk 2.14 adds a few new GtkAdjustment funcs, per diff below.

The configure one may be important, as it looks like you're forced to use that, as doing the right thing yourself with notifies produces a slew of superfluous changed signals.  (Which should be a bug, or a serious misfeature, as it can only slow down correct programs to attempt to help incorrect programs.)

The "set_" funcs check for an unchanged value (but perhaps wrongly comparing as gdoubles instead of gfloats) so I suppose should be included.  All those specific set functions offer type safety for C, but are really just bloat for perl :-(.
Comment 1 Torsten Schoenfeld 2010-01-09 19:34:08 UTC
The patch looks fine to me in general.  But I wonder if people might not expect the getters to be there, too.  Someone following the C API docs might expect to be able to call get_value(), for example.  So, should we just wrap these too?
Comment 2 Kevin Ryde 2010-01-14 00:27:34 UTC
I believe the gets add nothing at all to the present field accessors.  Most of the separate C get funcs are unnecessary in a dynamic language like perl, but those ones are particularly useless and I would leave them out for the time being.
Comment 3 Torsten Schoenfeld 2010-04-20 21:38:01 UTC
OK, you convinced.  Committed to master.