[PATCH] xf86: xf86ClearEntityListForScreen should take a pScrn

Alan Coopersmith alan.coopersmith at oracle.com
Thu May 24 17:03:23 PDT 2012


On 05/24/12 12:14 PM, Dave Airlie wrote:
> From: Dave Airlie <airlied at redhat.com>
> 
> When adding GPU screens this make life easier.
> 
> Signed-off-by: Dave Airlie <airlied at redhat.com>

Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

but you'd earn bonus points for also fixing the comment above the
function definition to have the right name for it:

/*
 * xf86ClearEntitiesForScreen() - called when a screen is deleted
 * to mark it's entities unused. Called by xf86DeleteScreen().
 */
void
xf86ClearEntityListForScreen(int scrnIndex)
{

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list