[PATCH 1/2] xfree86: vgaarb: simplify the arguments passed to lock/unlock

Keith Packard keithp at keithp.com
Fri May 21 12:46:56 PDT 2010


On Fri, 21 May 2010 20:49:06 +0300, "Vignatti Tiago (Nokia-D/Helsinki)" <tiago.vignatti at nokia.com> wrote:

> You gave the nak for the wrong patch from the series, right? First patch is 
> only a cleanup.

No, it's not a cleanup, it's an API change which passes the job of
computing the argument into the lower level function. Using a nice typed
pointer is "safer" than passing a bare int. The motivation for this
change was to change the implementation to use the Screen instead of the
ScrnInt, although I'm not sure why you want to make this change.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100521/eff47d40/attachment.pgp>


More information about the xorg-devel mailing list