2011/1/27 Erkki Seppälä <[email protected]>: > Removed unused assignment > > Reviewed-by: Ander Conselvan de Oliveira > <[email protected]> > Signed-off-by: Erkki Seppälä <[email protected]> > --- > 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; > } > -- > 1.7.0.4
I'd think the reason the code is like this is to avoid warning messages like warning: ignoring return value of ‘fgets’, declared with attribute warn_unused_result So, this may not be what you want to do. Matt _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
