[PATCH v2 03/25] [libx11] Removed unused assignment

Alan Coopersmith alan.coopersmith at oracle.com
Mon Jan 31 19:24:26 PST 2011


On 01/31/11 04:01 AM, Erkki Seppälä wrote:
> Pointer "pBuf" returned from "fgets(buf, 256, stream)" is never used
> 
> Reviewed-by: Ander Conselvan de Oliveira <ander.conselvan-de-oliveira at nokia.com>
> Signed-off-by: Erkki Seppälä <erkki.seppala at vincit.fi>
> ---
>  src/xcms/cmsColNm.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/xcms/cmsColNm.c b/src/xcms/cmsColNm.c
> index c7ad4fd..8518adf 100644
> --- a/src/xcms/cmsColNm.c
> +++ b/src/xcms/cmsColNm.c
> @@ -651,7 +651,7 @@ ReadColornameDB(
>       * Process lines between START_TOKEN to END_TOKEN
>       */
>  
> -    while ((pBuf = fgets(buf, XCMSDB_MAXLINELEN, stream)) != NULL) {
> +    while ((fgets(buf, XCMSDB_MAXLINELEN, stream)) != NULL) {
>  	if ((sscanf(buf, "%s", token)) && (strcmp(token, END_TOKEN) == 0)) {
>  	    /*
>  	     * Found END_TOKEN so break out of for loop

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

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list