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#comment797

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.

_______________________________________________
sr-dev mailing list
[email protected]
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to