On 27.01.2011 18:32, Matt Turner wrote:
-    while((pBuf = fgets(buf, XCMSDB_MAXLINELEN, stream)) != NULL) {
+    while((fgets(buf, XCMSDB_MAXLINELEN, stream)) != NULL) {

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.

But, its result is used, when it's compared against NULL?

(But the parenthesis that were left there after the fix seem superfluous.)
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to