[PATCH 3/5] Xext: remove needless /* parameter */ comments in declaration

Alan Coopersmith alan.coopersmith at oracle.com
Mon Mar 12 17:10:39 PDT 2012


On 03/12/12 04:20 PM, Peter Hutterer wrote:
> +    CARD64 inital_value,

Should that be "initial"?

> +    CARD64 resolution,
> +    SyncCounterType counterType,
>       SyncSystemCounterQueryValue QueryValue,
>       SyncSystemCounterBracketValues BracketValues
>   );
>
>   extern void SyncChangeCounter(
> -    SyncCounter *	/* pCounter*/,
> -    CARD64  		/* new_value */
> +    SyncCounter *  pCounte,
> +    CARD64  new_value

Shouldn't you just merge patch 4 with this instead of introducing a typo in
one patch and fixing in the next?

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


More information about the xorg-devel mailing list