Bug 51001 - patch for hook ap_lua_run_lua_open
Summary: patch for hook ap_lua_run_lua_open
Status: NEEDINFO
Alias: None
Product: Apache httpd-2
Classification: Unclassified
Component: mod_lua (show other bugs)
Version: 2.4-HEAD
Hardware: PC All
: P2 normal (vote)
Target Milestone: ---
Assignee: Apache HTTPD Bugs Mailing List
URL:
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2011-03-31 02:21 UTC by zhiguo zhao
Modified: 2016-11-23 03:27 UTC (History)
1 user (show)



Attachments
patch for hook ap_lua_run_lua_open (429 bytes, patch)
2011-03-31 02:21 UTC, zhiguo zhao
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description zhiguo zhao 2011-03-31 02:21:32 UTC
Created attachment 26819 [details]
patch for hook ap_lua_run_lua_open

This patch will make lua_open hooks available, that's important to extend moa_lua lua functions provider by other modules.

patch is very clear, please look attach file.
Comment 1 Jim Jagielski 2016-11-22 20:31:45 UTC
Why the removal of:

   lua_open_callback(L, p, NULL);

??
Comment 2 zhiguo zhao 2016-11-23 03:27:25 UTC
lua_open_callback should be remove from lua_open_hook, because lua_open_callback be called in vm_construct().

1. create_vm_spec set lua_open_callback to spec->cb;
2. vm_construct() call spec->cb;