Complete.Org: Mailing Lists: Archives: freeciv-dev: January 2004:
[Freeciv-Dev] Re: (PR#7251) new macros native_pos <-> index
Home

[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: jdorje@xxxxxxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#7251) new macros native_pos <-> index
From: "Raimar Falke" <i-freeciv-lists@xxxxxxxxxxxxx>
Date: Fri, 16 Jan 2004 15:55:24 -0800
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=7251 >

On Fri, Jan 16, 2004 at 12:03:49PM -0800, Jason Short wrote:
> 
> <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?

Do you really want another of this monster threads?

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "#!/usr/bin/perl -w
  if ( `date +%w` != 1 ) {
    die "This script only works on Mondays." ;
  }"
    -- from chkars.pl by Cornelius Krasel in de.comp.os.linux.misc




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