Re: [PATCH v4 1/1] Initial support for LuaJIT scripting

2017-06-26 Thread Dmitry V. Levin
On Mon, Jun 26, 2017 at 09:21:46PM +0300, Victor Krapivensky wrote: > > The patch renames one stringify macro and introduces two more temporary > > stringify macros. If this doesn't signal that we need a strace-wide > > stringify macro, then I don't know what does. > > OK. Should I put it into d

Re: [PATCH v4 1/1] Initial support for LuaJIT scripting

2017-06-26 Thread Victor Krapivensky
> The patch renames one stringify macro and introduces two more temporary > stringify macros. If this doesn't signal that we need a strace-wide > stringify macro, then I don't know what does. OK. Should I put it into defs.h, or create a new file for that? ---

Re: [PATCH v4 1/1] Initial support for LuaJIT scripting

2017-06-26 Thread Dmitry V. Levin
On Fri, Jun 23, 2017 at 06:05:34PM +0300, Victor Krapivensky wrote: [...] > --- /dev/null > +++ b/ffi.h > @@ -0,0 +1,18 @@ > +#ifndef STRACE_FFI_H > +#define STRACE_FFI_H > + > +#define FFI_STRINGIFY(...) #__VA_ARGS__ [...] > --- /dev/null > +++ b/luajit_lib.h > @@ -0,0 +1,224 @@ > +#define STRINGI

Re: [PATCH v4 1/1] Initial support for LuaJIT scripting

2017-06-26 Thread Dmitry V. Levin
On Fri, Jun 23, 2017 at 06:05:34PM +0300, Victor Krapivensky wrote: [...] > +#define MBRESTART(res, sig) > \ > + if ((res) >= 0 && ptrace_restart(PTRACE_SYSCALL, current_tcp, sig) < 0) > { \ > + /* Note: ptrace_restart