Date: Sun, 10 Jun 2001 10:49:02 +0300 (IDT)
From: Eli Zaretskii <eliz AT is DOT elta DOT co DOT il>
X-Sender: eliz AT is
To: Martin Str|mberg <ams AT ludd DOT luth DOT se>
cc: djgpp-workers AT delorie DOT com
Subject: Re: Compiler options for djdev build
In-Reply-To: <200106092056.WAA08741@mother.ludd.luth.se>
Message-ID: <Pine.SUN.3.91.1010610104847.5638D-100000@is>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Reply-To: djgpp-workers AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: djgpp-workers AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk


On Sat, 9 Jun 2001, Martin Str|mberg wrote:

> We're closing in. Some more hesitasion from me:

These all look fine to me.

> 
> Index: src/libc/compat/mntent/mntent.c
> ===================================================================
> RCS file: /cvs/djgpp/djgpp/src/libc/compat/mntent/mntent.c,v
> retrieving revision 1.6
> diff -p -u -r1.6 mntent.c
> --- src/libc/compat/mntent/mntent.c     2000/06/19 18:00:55     1.6
> +++ src/libc/compat/mntent/mntent.c     2001/06/09 20:50:23
> @@ -227,7 +227,7 @@ get_stacker_info(int drive_num)
>       illegal address abort when we try to peek at the signature below.
>       Actually, it's enough to test for (FFFFFh - 55h - drive), because
>       we need to get the host drive number at that offset. */
> -  if (stac_driver_ptr > 0x0000fffaa - drive_num)
> +  if (stac_driver_ptr > 0x0000fffaa - (unsigned)drive_num)
>      return 0;
>  
>    /* Stacker Anywhere returns pointer to 1 byte before the A55Ah
> Index: src/libc/dos/errno/doserr2e.c
> ===================================================================
> RCS file: /cvs/djgpp/djgpp/src/libc/dos/errno/doserr2e.c,v
> retrieving revision 1.3
> diff -p -u -r1.3 doserr2e.c
> --- src/libc/dos/errno/doserr2e.c       2001/02/01 19:48:07     1.3
> +++ src/libc/dos/errno/doserr2e.c       2001/06/09 20:50:24
> @@ -44,7 +44,7 @@ int
>  __doserr_to_errno(int doserr)
>  {
>    _doserrno = doserr;
> -  if (doserr >= 0 && doserr < sizeof(map)/sizeof(map[0]))
> +  if (doserr >= 0 && (unsigned int)doserr < sizeof(map)/sizeof(map[0]))
>      return map[doserr];
>    return EINVAL;
>  }
> Index: src/libc/dos/io/dosio.c
> ===================================================================
> RCS file: /cvs/djgpp/djgpp/src/libc/dos/io/dosio.c,v
> retrieving revision 1.1
> diff -p -u -r1.1 dosio.c
> --- src/libc/dos/io/dosio.c     1995/11/25 02:06:10     1.1
> +++ src/libc/dos/io/dosio.c     2001/06/09 20:50:25
> @@ -38,7 +38,7 @@ __file_handle_set(int fd, int mode)
>  
>    /* See if we need to expand the tables.  Check this BEFORE it might fail,
>       so that when we hit the count'th request, we've already up'd it. */
> -  if (fd >= (count-1) && count < 255)
> +  if ((size_t)fd >= (count-1) && count < 255)
>    {
>      int oldcount = count;
>      count = 255;
> 
> 
> Right,
> 
> 						MartinS
>