Re: [ANN] patch-lua-5.1.4-1

[Dirk Feytons <dirk.feytons@...>]

On Thu, Aug 12, 2010 at 22:16, nick ralabate <ralabate@gmail.com> wrote:
> Hi, was this question ever answered about the 5.1.4 patch?  Like OP, I
> am mainly curious.
>
>
> -Nick
>
>
>
> Sean Connor said:
>
>>It was thus said that the Great Luiz Henrique de Figueiredo once stated:
>>>
>>> All feedback welcome.
>
>>  What bug does the following fix?
>
>>--- loadlib.c  2008/08/06 13:29:28     1.52.1.3
>>+++ loadlib.c  2009/09/09 13:17:16     1.52.1.4
>>@@ -1,5 +1,5 @@
>  >/*
>>-** $Id: loadlib.c,v 1.52.1.3 2008/08/06 13:29:28 roberto Exp $
>>+** $Id: loadlib.c,v 1.52.1.4 2009/09/09 13:17:16 roberto Exp $
>> ** Dynamic library loader for Lua
>> ** See Copyright Notice in lua.h
>> **
>>@@ -639,7 +639,7 @@
>>   lua_pushvalue(L, -1);
>>   lua_replace(L, LUA_ENVIRONINDEX);
>>   /* create oaders' table */
>>-  lua_createtable(L, 0, sizeof(loaders)/sizeof(loaders[0]) - 1);
>>+  lua_createtable(L, sizeof(loaders)/sizeof(loaders[0]) - 1, 0);
>>   /* fill it with pre-defined loaders */
>>  for (i=0; loaders[i] != NULL; i++) {
>>     lua_pushcfunction(L, loaders[i]);
>>
>>  I don't see a description on http://www.lua.org/bugs.html#5.1.4 for it.
>>
>>  -spc (Just curious ... )

See http://lua-users.org/lists/lua-l/2009-09/msg00072.html
It's not really a bug; just a mere inefficiency.

-- 
Dirk