delorie.com/archives/browse.cgi | search |
Mailing-List: | contact cygwin-help AT sourceware DOT cygnus DOT com; run by ezmlm |
List-Subscribe: | <mailto:cygwin-subscribe AT sources DOT redhat DOT com> |
List-Archive: | <http://sources.redhat.com/ml/cygwin/> |
List-Post: | <mailto:cygwin AT sources DOT redhat DOT com> |
List-Help: | <mailto:cygwin-help AT sources DOT redhat DOT com>, <http://sources.redhat.com/ml/#faqs> |
Sender: | cygwin-owner AT sources DOT redhat DOT com |
Delivered-To: | mailing list cygwin AT sources DOT redhat DOT com |
From: | "Christian Jullien" <eligis AT wanadoo DOT fr> |
To: | <cygwin AT sources DOT redhat DOT com> |
Subject: | Re: test report on ver 1.1.5-4 |
Date: | Sat, 4 Nov 2000 18:31:05 +0100 |
Message-ID: | <IMEDKJHLFGIGPDAKFBKGCEMMCBAA.eligis@wanadoo.fr> |
MIME-Version: | 1.0 |
X-Priority: | 3 (Normal) |
X-MSMail-Priority: | Normal |
X-Mailer: | Microsoft Outlook IMO, Build 9.0.2416 (9.0.2910.0) |
Importance: | Normal |
X-MimeOLE: | Produced By Microsoft MimeOLE V5.50.4133.2400 |
> * The FD_ZERO() macro produces the following statement: > warning: comparison between signed and unsigned In my previous mail 'PROPOSED FIX to remove warning in 1.1.5-3 sys/types.h' I found exactly the same warning and propose a solution. Maybe -ansi or even -pedantic will raise that warning. Here is a copy of my previous mail. --- copy --- # define FD_ZERO(p) (__extension__ (void)({ \ int i; \ char *__tmp = (char *)p; \ for (i = 0; i < sizeof (*(p)); ++i) \ *__tmp++ = 0; \ })) raise a new warning 'signed unsigned comparison' I suggest to replace int i; with unsigned int i; A better solution should be to include stddef (for size_t) and have: size_t i; which is the type returned by sizeof. But, unless I'm totaly wrong, the fastest solution is to use memset that will be expanded inline with assembly code. Right? Christian Jullien -- Want to unsubscribe from this list? Send a message to cygwin-unsubscribe AT sourceware DOT cygnus DOT com
webmaster | delorie software privacy |
Copyright © 2019 by DJ Delorie | Updated Jul 2019 |