Hi Jim,

> FYI, without this patch and with very recent gcc, I would see warnings
> like this:
>
> lib/freading.c:29:1: error: function might be candidate for attribute 'pure'\
> [-Werror=suggest-attribute=pure]

The documentation [1] mentions as example of a non-pure function "feof in a
multithreaded environment". freading is similar to feof. But our extended stdio
functions have little chance of being useful in an environment where the same
stream is accessed from different threads. Therefore OK for the change.

But all of the functions
  freadable
  freadahead
  freading
  fwritable
  fwriting
are similar enough that they should be treated the same way. Can you add the
'__pure__' attribute also the other 4 functions as well?

Bruno

[1] http://gcc.gnu.org/onlinedocs/gcc/Function-Attributes.html

Reply via email to