MINOR/DOC: lua: just precise one thing
In the case of applet, the Lua context is taken from session
when we get the private values. This patch just update comments
associated to this action because it is not obvious.
diff --git a/src/hlua.c b/src/hlua.c
index 2621ba6..f662cb0 100644
--- a/src/hlua.c
+++ b/src/hlua.c
@@ -3298,7 +3298,10 @@
{
struct hlua_appctx *appctx = MAY_LJMP(hlua_checkapplet_tcp(L, 1));
struct stream *s = appctx->htxn.s;
- struct hlua *hlua = &s->hlua;
+ struct hlua *hlua;
+
+ /* Note that this hlua struct is from the session and not from the applet. */
+ hlua = &s->hlua;
MAY_LJMP(check_args(L, 2, "set_priv"));
@@ -3317,7 +3320,10 @@
{
struct hlua_appctx *appctx = MAY_LJMP(hlua_checkapplet_tcp(L, 1));
struct stream *s = appctx->htxn.s;
- struct hlua *hlua = &s->hlua;
+ struct hlua *hlua;
+
+ /* Note that this hlua struct is from the session and not from the applet. */
+ hlua = &s->hlua;
/* Push configuration index in the stack. */
lua_rawgeti(L, LUA_REGISTRYINDEX, hlua->Mref);
@@ -3746,7 +3752,10 @@
{
struct hlua_appctx *appctx = MAY_LJMP(hlua_checkapplet_http(L, 1));
struct stream *s = appctx->htxn.s;
- struct hlua *hlua = &s->hlua;
+ struct hlua *hlua;
+
+ /* Note that this hlua struct is from the session and not from the applet. */
+ hlua = &s->hlua;
MAY_LJMP(check_args(L, 2, "set_priv"));
@@ -3765,7 +3774,10 @@
{
struct hlua_appctx *appctx = MAY_LJMP(hlua_checkapplet_http(L, 1));
struct stream *s = appctx->htxn.s;
- struct hlua *hlua = &s->hlua;
+ struct hlua *hlua;
+
+ /* Note that this hlua struct is from the session and not from the applet. */
+ hlua = &s->hlua;
/* Push configuration index in the stack. */
lua_rawgeti(L, LUA_REGISTRYINDEX, hlua->Mref);
@@ -6693,8 +6705,6 @@
/* Once the arguments parsed, the CLI is like an AppletTCP,
* so push AppletTCP in the stack.
- * TODO: get_priv() and set_priv() are useless. Maybe we will
- * create a new object without these two functions.
*/
if (!hlua_applet_tcp_new(hlua->T, appctx)) {
SEND_ERR(NULL, "Lua cli '%s': full stack.\n", fcn->name);