Complete.Org: Mailing Lists: Archives: freeciv-dev: February 2004:
[Freeciv-Dev] (PR#6822) some variable-sized arrays
Home

[Freeciv-Dev] (PR#6822) some variable-sized arrays

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: undisclosed-recipients: ;
Subject: [Freeciv-Dev] (PR#6822) some variable-sized arrays
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Thu, 26 Feb 2004 17:23:32 -0800
Reply-to: rt@xxxxxxxxxxx

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

> [i-freeciv-lists@xxxxxxxxxxxxx - Tue Nov 11 06:59:41 2003]:
> 
> On Mon, Nov 10, 2003 at 04:56:27PM -0800, Jason Short wrote:
> > 
> > <URL: http://rt.freeciv.org/Ticket/Display.html?id=6822 >
> > 
> > The attached patch replaces some malloc'd arrays with variable-sized 
> > stack arrays.  One memory leak is fixed in the process.
> > 
> > This should be faster, but my tests didn't show a measurable difference 
> > (on systems with inefficient malloc calls this may make an impact - but 
> > then those systems aren't likely to compile C99 anyway).  It's also 
> > safer and more legible.
> 
> I don't see that the benefits (safer) outweigh the costs (change must
> be done, compiler or other tools doesn't understand it).

Do you still object to the patch?  Can I apply it?

jason



[Prev in Thread] Current Thread [Next in Thread]
  • [Freeciv-Dev] (PR#6822) some variable-sized arrays, Jason Short <=