[Tarantool-patches] [PATCH luajit] tools: make memprof parser output user-friendly

Igor Munkin imun at tarantool.org
Mon Mar 29 22:11:26 MSK 2021


Sergey,

On 29.03.21, Sergey Kaplun wrote:
> Igor,
> 

<snipped>

> 
> Thanks! I like it, exept some typos:
> 
> > 
> > | ##feature/luajit
> > |
> > | * Make LuaJIT memory profiler parser output more user-friendly (gh-5811).
> > |   Now the source line definition where the event occurs is much clearer:
> > |   only source file name and allocation-related line are presented,
> 
> Typo? s/are presented/is presented/

No, since both *are* presented.

> 
> > |   function definition line number is omitted. Moreover, event-related
> 
> Typo: /function definition line number/the function definition line number/.

Fixed.

> 
> > |   statistics are indicated with units.
> > |   **Breaking change**: Line info of the line function definition is
> > |   saved in symbol info table by field `linedefined` now and field `name`
> > |   is renamed to `source` with the respect to Lua Debug API.
> 
> Typo: s/to Lua Debug/to the Lua Debug/

Fixed.

> 

<snipped>

> 
> -- 
> Best regards,
> Sergey Kaplun

-- 
Best regards,
IM


More information about the Tarantool-patches mailing list