Am 11.07.2009, 11:45 Uhr, schrieb Stephen R. van den Berg <srb@xxxxxxx>:
Matthias Andree wrote:
+ /* do not optimize the next two lines - they must both be executed in
+ * this order. || might short-circuit the fclose(), and combining them
+ * into one statement might reverse the order of execution.
+ * Also, fflush() may not be sufficient - on some file systems, the
+ * error is still delayed until the final [f]close(). */
+ e = ferror(f);
+ e |= fclose(f);
+ if (e)
The commentary above should be common knowledge for anyone familiar with
ANSI C. So I'd suggest moving the comments into the description section
of
the commit and removing them from the actual code.
Feel free to do it and submit a patch, I'm not going to invest more time
into a piece of code that runs seldomly.
--
Matthias Andree
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html