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 605199 - Contradiction in the description of function gtk_container_set_border_width() and property 'border-width'
Contradiction in the description of function gtk_container_set_border_width()...
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: Other
2.90.x
Other All
: Normal normal
: 3.0
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2009-12-22 07:44 UTC by tigran
Modified: 2010-09-08 23:27 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch to the 'border-width' property (68 bytes, text/plain)
2009-12-22 07:44 UTC, tigran 		  Details
Limit GtkContainer::border-width to 65535 (608 bytes, patch)
2010-01-11 09:26 UTC, Christian Dywan 		committed Details | Review

Description tigran 2009-12-22 07:44:31 UTC 
Created attachment 150219 [details]
patch to the 'border-width' property

It is written in the description of the function gtk_container_set_border_width() that valid values for the second argument of the function are in the range 0-65535, but in the description of the property 'border-width' is written that allowed values are <= G_MAXINT. And indeed we can't set to the property 'border-width' value bigger than 65535, it'll be truncated to 16 bit value


Also reported at http://linuxtesting.org/results/report?num=D0145
Comment 1 Christian Dywan 2010-01-05 09:32:25 UTC 
This is an ABI change, not just documentation. We should probably do this in 2.9.
Comment 2 Matthias Clasen 2010-01-07 14:33:39 UTC 
Being able to set a widgets border to 2^21 is not in any way a relevant ABI thats worth preserving.
Comment 3 Christian Dywan 2010-01-11 09:26:08 UTC 
Created attachment 151162 [details] [review]
Limit GtkContainer::border-width to 65535
Comment 4 Christian Dywan 2010-01-11 16:37:33 UTC 
Pushed to master.