On Jun 26, 2007, at 3:18 PM, Ito Kazumitsu wrote:

I am afraild Casey Marshall's comment could not reach me.
I have just found this comment in the mailing list archive.


Your mail host dislikes it when someone from gnu.org sends mail from dreamhost.com. So it goes.

Casey Marshall wrote.

| Not sure why you have this block...
|
| > +#if defined(ENOTTY) && defined(HAVE_FSTAT)
| > +      if (errno == ENOTTY)
| > +        {
| > +          if ((fstat (fd, &statBuffer) == 0) && S_ISREG
| > (statBuffer.st_mode))
| > +            {
| > +              n = lseek (fd, 0, SEEK_CUR);
| > +              if (n != -1)
| > +                {
| > +                  avail = statBuffer.st_size - n;
| > +                  return avail;
| > +                }
| > +            }
| > +        }
| > +#endif
| > +      JCL_ThrowException (env, IO_EXCEPTION, strerror (errno));
|
| ...surrounding it, though.

I do not like this surrounding "#if", but I am afraid in some environment
defined(HAVE_FSTAT) can be false.


I meant the curly braces, the #if..#endif is appropriate.

Minor issue, of course.

Thanks.

Reply via email to