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 512860 - add macro for packed attribute
add macro for packed attribute
Status: RESOLVED WONTFIX
Product: glib
Classification: Platform
Component: general
unspecified
Other All
: Normal enhancement
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2008-01-29 18:30 UTC by Christian Persch
Modified: 2013-02-03 15:44 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
patch (1.17 KB, patch)
2008-01-29 18:34 UTC, Christian Persch
none Details | Review
updated patch to master (1.91 KB, patch)
2009-05-19 12:01 UTC, Christian Persch
none Details | Review
updated patch to master (1.91 KB, patch)
2010-01-08 18:17 UTC, Christian Persch
none Details | Review

Description Christian Persch 2008-01-29 18:30:50 UTC
Like all the other gcc attributes, G_GNUC_PACKED for __attribute__((packed)).
Comment 1 Christian Persch 2008-01-29 18:34:49 UTC
Created attachment 103977 [details] [review]
patch

http://www.ohse.de/uwe/articles/gcc-attributes.html#type-packed says the attribute exists since gcc 2.7.
Comment 2 Christian Persch 2009-05-19 12:01:16 UTC
Created attachment 134925 [details] [review]
updated patch to master
Comment 3 Matthias Clasen 2009-05-26 05:03:01 UTC
Do you have any concrete uses for this macro ? I don't think we want to add wrappers for random gcc attributes unless there is a concrete use case.
Comment 4 Christian Persch 2009-05-26 09:10:02 UTC
I filed this bug when I had need of the packed attribute in bug 512799 for gnome-games.
Comment 5 Christian Persch 2009-05-26 09:27:36 UTC
A quick grep over my gnome checkout finds the packed attribute also used in NetworkManager and Rhythmbox.
Comment 6 Christian Persch 2010-01-08 18:17:06 UTC
Created attachment 151047 [details] [review]
updated patch to master

Any chance to get this in for 2.24 ?