THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
The following task has a new comment added:
FS#270 - app_lua: add rpc reload command
User who did this - Daniel-Constantin Mierla (miconda)
----------
It doesn't seem that sr_lua_script_ver->len needs to be protected by a lock, it is
set once, outside of any lock, at init. Then you read it inside a lock, but no other
possibility to be updated at the same time.
Checking if there is need to reload scripts every time a function is executed seems a bit
too time consuming from my point of view. Perhaps there can be added some optimization,
but that can be subject to future improvements. For the beginning it would be good to have
a module parameter that enables/disables the reloading. It this way one that does not need
reloading does not have to be exposed to the loops executed to detect and reload scripts
for each function execution.
----------
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=270#comment7…
You are receiving this message because you have requested it from the Flyspray bugtracking
system. If you did not expect this message or don't want to receive mails in future,
you can change your notification settings at the URL shown above.