Complete.Org: Mailing Lists: Archives: freeciv-dev: December 2005:
[Freeciv-Dev] Re: (PR#14764) Patch to get SDL client compile and start o
Home

[Freeciv-Dev] Re: (PR#14764) Patch to get SDL client compile and start o

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Subject: [Freeciv-Dev] Re: (PR#14764) Patch to get SDL client compile and start on AmigaOS
From: "Nicolas A. Mendoza" <mendoza@xxxxxxxxxxx>
Date: Sun, 4 Dec 2005 11:39:12 -0800
Reply-to: bugs@xxxxxxxxxxx

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=14764 >

yOn Sun, 4 Dec 2005, Vasco Alexandre da Silva Costa wrote:

>
> <URL: http://bugs.freeciv.org/Ticket/Display.html?id=14764 >
>
> On 12/4/05, Nicolas A. Mendoza <mendoza@xxxxxxxxxxx> wrote:
>>
>> <URL: http://bugs.freeciv.org/Ticket/Display.html?id=14764 >
>
> Hello Nicolas,
>
>> - renamed "help" enum's members to HELP_#? (to avoid clash with TEXT)
>
> Seems fine.
>
>> - Added an Amiga entry in configure.ac to allow run as root (only one user
>> in AmigaOS currently)
>
> Looks fine.
>
>> - utility/mem.c relies on that malloc(0) returns a pointer with 0 or as
>> little as possible mem allocated. This is implementation specific
>> according to ANSI C, and with the c library clib2 on AmigaOS, it returns
>> an as correct NULL pointer isntead. Quick #ifdef hack that always
>> allocates at least one byte. (since the poitner is always going to be
>> freed anyway)
>
> Ah, I would rather make it always like this, than adding an OS specific 
> #ifdef.
> You could just do:
>  if (size == 0) {
>    size = 1;
>  }
>
> -Vasco Alexandre da Silva Costa
>
>

Fixed and added comment.

http://home.poalrboing.com/nicomen/freeciv/freeciv-nicomen_20051205.patch.gz

-- 
Nicolas Mendoza

Attachment: freeciv-nicomen_20051205.patch.gz
Description: Zip compressed data


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