Fix failure

bin/fdutils_macos.cc

Index: bin/fdutils_macos.cc
===================================================================
--- bin/fdutils_macos.cc	(revision 3211)
+++ bin/fdutils_macos.cc	(working copy)
@@ -39,12 +39,12 @@
 
 
 intptr_t FDUtils::AvailableBytes(intptr_t fd) {

[Søren Gjesse, 2012/01/12 07:32:47]

I have it on my TODO list to change all these uses of intptr_t in system calls
to the types actually used in the .h files where the calls are declared.

[siva, 2012/01/12 19:30:50]

Ok.

On 2012/01/12 07:32:47, Søren Gjesse wrote:

-  size_t available;
+  int32_t available;

[Søren Gjesse, 2012/01/12 07:32:47]

Shouldn't we just use int here as that is what is specified on the ioctl_list
man page?

[siva, 2012/01/12 19:30:50]

I would prefer int32_t as opposed to int as it makes it clear that we are
dealing with a 32 bit quantity here.
But I agree you are worried if the kernel has a different interpretation of
'int' from 'int32_t' this would break again.
Changed to use 'int'


On 2012/01/12 07:32:47, Søren Gjesse wrote:

   int result = TEMP_FAILURE_RETRY(ioctl(fd, FIONREAD, &available));
   if (result < 0) {
     return result;
   }
-  return available;
+  return static_cast<intptr_t>(available);
 }