[Freeciv-Dev] Re: (PR#7251) new macros native_pos <-> index
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
|
To: |
undisclosed-recipients: ; |
|
Subject: |
[Freeciv-Dev] Re: (PR#7251) new macros native_pos <-> index |
|
From: |
"Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx> |
|
Date: |
Fri, 16 Jan 2004 12:03:49 -0800 |
|
Reply-to: |
rt@xxxxxxxxxxx |
<URL: http://rt.freeciv.org/Ticket/Display.html?id=7251 >
Raimar Falke wrote:
> <URL: http://rt.freeciv.org/Ticket/Display.html?id=7251 >
>
> On Thu, Jan 15, 2004 at 09:02:11PM -0800, Jason Short wrote:
>
>><URL: http://rt.freeciv.org/Ticket/Display.html?id=7251 >
>>
>>The attached patch adds macros native_pos_to_index and index_to_native_pos.
>>
>>These accomplish two things:
>>
>>- They make the steps in the other macros more explicit.
>>- They will be of use in future functions that access values by native
>>position.
>>
>>Because of their extremely heavy use they obviously have to be macros
>>(or inline functions).
>
>
> Shouldn't these be inline functions?!
Um, should they be? Why?
jason
|
|
