httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <>
Subject Re: cvs commit: apache-apr/include apr_file_io.h
Date Fri, 09 Apr 1999 21:46:16 GMT wrote:
> rbb         99/04/09 11:02:59
>   Modified:    apr/file_io/unix readwrite.c
>                apr/test testfile.c
>                docs     fileio.txt
>                include  apr_file_io.h
>   Log:
>   Added apr_writev call, and requisite defines.  I also added the permissions
>   abstraction #defines.
>   -
>   -apr_ssize_t apr_read(apr_file_t *thefile, void *buf, apr_size_t nbytes)
>   +#include <sys/uio.h>
>   +apr_ssize_t apr_read(apr_file_t *thefile, void *buf, apr_ssize_t nbytes)

I think this is wrong. On my Linux box here, I see that "nbytes" is
listed as a size_t, NOT an ssize_t. It does *return* and ssize_t,

>   +apr_ssize_t apr_writev(apr_file_t *thefile, const apr_iovec_t * vec, apr_ssize_t iocnt)

On my system, that last parameter is an integer count (of iovec
structs), not an ssize_t.

IMO, these parameters should be changed to an ssize_t and an int until
another platform is found which defines them differently. At that point,
a choice will need to be made.


Greg Stein,

View raw message