You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by jp...@apache.org on 2012/06/12 17:51:36 UTC

[11/50] git commit: More comments

More comments


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/9044f9d1
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/9044f9d1
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/9044f9d1

Branch: refs/heads/jpeach/lua
Commit: 9044f9d1e511a47a4c0a8182ea585ad774442400
Parents: f69e100
Author: James Peach <jp...@apache.org>
Authored: Thu Apr 19 21:18:10 2012 -0700
Committer: James Peach <jp...@apache.org>
Committed: Tue Jun 12 08:48:37 2012 -0700

----------------------------------------------------------------------
 plugins/lua/lua.cc |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/9044f9d1/plugins/lua/lua.cc
----------------------------------------------------------------------
diff --git a/plugins/lua/lua.cc b/plugins/lua/lua.cc
index f967b35..b13537c 100644
--- a/plugins/lua/lua.cc
+++ b/plugins/lua/lua.cc
@@ -111,6 +111,8 @@ LuaPluginRemap(lua_State * lua, TSHttpTxn txn, TSRemapRequestInfo * rri)
 
   // XXX The 'to' and 'from' URLs are supposed to be static so we can cache them somewhere
   // in the Lua state.
+  // XXX Do we really need to pass this at all? You can get the current URL from the request, so
+  // there's not really a need for these that I can see ...
   LuaPushUrl(lua, rri->requestBufp, rri->mapFromUrl);
   LuaPushUrl(lua, rri->requestBufp, rri->mapToUrl);