[Freeciv-Dev] (PR#6822) some variable-sized arrays
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
<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 <=
|
|