[PATCH 0/7] Resume breaking ABI
Keith Packard
keithp at keithp.com
Fri Oct 5 14:21:47 PDT 2012
Colin Harrison <colin.harrison at virgin.net> writes:
> Hi,
>
> Thinking about this after strong coffee: all those bit-fields would be
> better unsigned...
Yeah, I agree -- Adam, what do you think?
> --- ./include/save_dixstruct.h 2012-10-05 01:24:03.981982326 +0100
> +++ ./include/dixstruct.h 2012-10-05 20:13:19.803253520 +0100
> @@ -90,12 +90,12 @@ typedef struct _Client {
> Mask clientAsMask;
> short index;
> unsigned char majorOp, minorOp;
> - int swapped:1;
> - int local:1;
> - int big_requests:1; /* supports large requests */
> - int clientGone:1;
> - int closeDownMode:2;
> - int clientState:2;
> + unsigned swapped:1;
> + unsigned local:1;
> + unsigned big_requests:1; /* supports large requests */
> + unsigned clientGone:1;
> + unsigned closeDownMode:2;
> + unsigned clientState:2;
> char smart_priority;
> short noClientException; /* this client died or needs to be killed
> */
> int priority;
I'll need a signed-off-by: line at least; a regular git patch would be
even nicer.
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20121005/676b4a07/attachment.pgp>
More information about the xorg-devel
mailing list