You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by sf...@apache.org on 2011/07/17 18:33:04 UTC
svn commit: r1147680 - /httpd/httpd/trunk/modules/lua/lua_vmprep.c
Author: sf
Date: Sun Jul 17 16:33:03 2011
New Revision: 1147680
URL: http://svn.apache.org/viewvc?rev=1147680&view=rev
Log:
Simplify code by using apr_array_pstrcat()
Modified:
httpd/httpd/trunk/modules/lua/lua_vmprep.c
Modified: httpd/httpd/trunk/modules/lua/lua_vmprep.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/lua/lua_vmprep.c?rev=1147680&r1=1147679&r2=1147680&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/lua/lua_vmprep.c (original)
+++ httpd/httpd/trunk/modules/lua/lua_vmprep.c Sun Jul 17 16:33:03 2011
@@ -246,7 +246,6 @@ static void munge_path(lua_State *L,
const char *pattern;
const char *modified;
char *part;
- int i;
lua_getglobal(L, "package");
lua_getfield(L, -1, field);
@@ -259,11 +258,8 @@ static void munge_path(lua_State *L,
modified = lua_tostring(L, -1);
lua_pop(L, 2);
- part = apr_pstrdup(pool, modified);
- for (i = 0; i < paths->nelts; i++) {
- const char *new_path = ((const char **) paths->elts)[i];
- part = apr_pstrcat(pool, part, ";", new_path, NULL);
- }
+ part = apr_pstrcat(pool, modified, apr_array_pstrcat(pool, paths, ';'),
+ NULL);
lua_pushstring(L, part);
lua_setfield(L, -2, field);
lua_pop(L, 1); /* pop "package" off the stack */