Complete.Org: Mailing Lists: Archives: freeciv-dev: February 2002:
[Freeciv-Dev] Re: New Patch
Home

[Freeciv-Dev] Re: New Patch

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: "freeciv-dev@xxxxxxxxxxx" <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: New Patch
From: "Alexander Mai" <st002279@xxxxxxxxxxxxxxxxxxxxxx>
Date: Wed, 06 Feb 2002 18:12:09 +0100 (CET)
Reply-to: "Alexander Mai" <st002279@xxxxxxxxxxxxxxxxxxxxxx>

On Wed, 6 Feb 2002 16:19:06 +0100, Raimar Falke wrote:

>> There are indeed both kinds of warnings and my cast
>> to (long) is to be well justified.
>
>The original code works without warning as long as size_t isn't a
>long. There are no information about this in the C spec. To avoid this
>you can do a cast to int: "(int)sizeof(...)".
>

In turn your proposal would break if sizeof() is actually 
of type long. (and it can't be casted perhaps...).
My solution is perhaps not perfect, just "better" and for most
systems it is fine. Printing size_t is a famous programming FAQ.

You don't need to teach me C programming, guys.


---
Alexander Mai
st002279@xxxxxxxxxxxxxxxxxxxxxx




[Prev in Thread] Current Thread [Next in Thread]