On 2016-02-02 23:25, Paul B Mahol wrote:
> Hi,
> 
> patch attached.

Nice.  I look forward to reading it.

Firstly: why limit it to Lua 5.1?  I think it should also support
LuaJIT.  While it is ABI compatible with 5.1 this patch would require
its headers to be in "lua-5.1".  My suggestion would be to test for
LuaJIT first then fallback to 5.1.  (Perhaps it isn't doing a lot in Lua
code which then makes LuaJIT a little redundant.)

Anyway, that is getting a little off topic.  I will apply read the patch
in detail later.

Something else I should note.  If you don't take care to provide a
sandbox people could end up executing arbitrary code if they run scripts
from the net.


Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to