Mailing-List: contact cygwin-help AT sourceware DOT cygnus DOT com; run by ezmlm
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie DOT com AT sourceware DOT cygnus DOT com>
List-Subscribe: <mailto:cygwin-subscribe AT sourceware DOT cygnus DOT com>
List-Archive: <http://sourceware.cygnus.com/ml/cygwin/>
List-Post: <mailto:cygwin AT sourceware DOT cygnus DOT com>
List-Help: <mailto:cygwin-help AT sourceware DOT cygnus DOT com>, <http://sourceware.cygnus.com/ml/#faqs>
Sender: cygwin-owner AT sourceware DOT cygnus DOT com
Delivered-To: mailing list cygwin AT sourceware DOT cygnus DOT com
Message-ID: <3848C72F.2F05B5C0@cygnus.com>
Date: Fri, 03 Dec 1999 23:47:59 -0800
From: Eric Christopher <echristo AT cygnus DOT com>
X-Mailer: Mozilla 4.51 [en] (X11; I; Linux 2.2.5-15 i586)
X-Accept-Language: en
MIME-Version: 1.0
To: martin AT xemacs DOT org
CC: cygwin AT sourceware DOT cygnus DOT com, XEmacs NT <xemacs-nt AT xemacs DOT org>
Subject: Re: cygwin-1999-12-01: Prototypes in system headers not 
 standard-compliant
References: <14408 DOT 38135 DOT 146580 DOT 963528 AT lasker DOT 666 DOT com>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

I'll test these out.  Does anyone have a pointer to the full Unix 98 spec?

-eric

> The prototype for read() and write() in Unix98 don't match those in
> the Cygwin header files.
>
> http://www.opengroup.org/onlinepubs/007908799/xsh/read.html
> http://www.opengroup.org/onlinepubs/007908799/xsh/write.html
>
> The standard return type for these functions is `ssize_t'.
> The traditional return type for these functions is `int'.
> The return type in Cygwin is `int'.
>
> On 32-bit platforms, like Cygwin, the best choice is probably to use
> `ssize_t' in the prototypes (they are standard, after all, right?),
> but to then have
>
> typedef int ssize_t;
>
> This is effectively what Linux does.
>
> Here's the obvious untested patch (I've never used Cygwin):
>
> --- sys/unistd.h~       Wed Dec  1 21:25:48 1999
> +++ sys/unistd.h        Fri Dec  3 20:07:58 1999
> @@ -75,10 +75,10 @@
>  pid_t   _EXFUN(_getpid, (void ));
>  int     _EXFUN(_link, (const char *__path1, const char *__path2 ));
>  off_t   _EXFUN(_lseek, (int __fildes, off_t __offset, int __whence ));
> -int     _EXFUN(_read, (int __fildes, void *__buf, size_t __nbyte ));
> +ssize_t _EXFUN(_read, (int __fildes, void *__buf, size_t __nbyte ));
>  void *  _EXFUN(_sbrk,  (size_t __incr));
>  int     _EXFUN(_unlink, (const char *__path ));
> -int     _EXFUN(_write, (int __fildes, const void *__buf, size_t __nbyte ));
> +ssize_t _EXFUN(_write, (int __fildes, const void *__buf, size_t __nbyte ));
>
>  #if defined(__CYGWIN32__) || defined(__rtems__)
>  unsigned _EXFUN(usleep, (unsigned int __useconds));
>
> --- sys/types.h~        Wed Dec  1 21:25:48 1999
> +++ sys/types.h Fri Dec  3 20:09:24 1999
> @@ -110,7 +110,7 @@
>  typedef        unsigned short  gid_t;
>  typedef int pid_t;
>  typedef        long key_t;
> -typedef long ssize_t;
> +typedef int ssize_t;
>
>  #ifdef __MS_types__
>  typedef        char *  addr_t;
>
> --- sys/_types.h~       Wed Dec  1 21:25:42 1999
> +++ sys/_types.h        Fri Dec  3 20:09:10 1999
> @@ -10,6 +10,6 @@
>  #define _SYS__TYPES_H
>
>  typedef long _off_t;
> -typedef long _ssize_t;
> +typedef int _ssize_t;
>
>  #endif /* _SYS__TYPES_H */
>
> P.S. If you do this, check all other functions for Unix98-compliance.


--
Want to unsubscribe from this list?
Send a message to cygwin-unsubscribe AT sourceware DOT cygnus DOT com