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

Alan Coopersmith alan.coopersmith at oracle.com
Mon Jan 31 19:23:27 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 4164370..c7ad4fd 100644
> --- a/src/xcms/cmsColNm.c
> +++ b/src/xcms/cmsColNm.c
> @@ -567,7 +567,7 @@ stringSectionSize(
>  	return(XcmsFailure);
>      }
>  
> -    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)) {
>  	    break;
>  	}

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