Skip to content

LuaJIT memory profiler documentation #2085

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Apr 29, 2021
Merged

Conversation

veod32
Copy link
Collaborator

@veod32 veod32 commented Apr 20, 2021

No description provided.

@veod32 veod32 requested review from igormunkin and Buristan April 20, 2021 18:35
@veod32 veod32 self-assigned this Apr 20, 2021
@veod32 veod32 linked an issue Apr 20, 2021 that may be closed by this pull request
@github-actions github-actions bot temporarily deployed to branch-gh-1753-luajit-profiler April 20, 2021 18:37 Inactive
@github-actions github-actions bot temporarily deployed to branch-gh-1753-luajit-profiler April 20, 2021 19:01 Inactive
@github-actions github-actions bot temporarily deployed to branch-gh-1753-luajit-profiler April 21, 2021 04:59 Inactive
@github-actions github-actions bot temporarily deployed to branch-gh-1753-luajit-profiler April 21, 2021 11:19 Inactive
Copy link

@Buristan Buristan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Thanks for the great work!

Answered on your questions below.

@github-actions github-actions bot temporarily deployed to branch-gh-1753-luajit-profiler April 26, 2021 16:30 Inactive
Copy link
Contributor

@igormunkin igormunkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@veod32, thanks for the patch! Though I left some comments, the doc is neat. Let's fix the remaining and push it!

@veod32 veod32 force-pushed the veod32/gh-1753-luajit-profiler branch 2 times, most recently from c1a116f to dbd6b87 Compare April 28, 2021 14:24
@NickVolynkin NickVolynkin force-pushed the veod32/gh-1753-luajit-profiler branch from e542d21 to dbd6b87 Compare April 28, 2021 14:32
@NickVolynkin NickVolynkin force-pushed the veod32/gh-1753-luajit-profiler branch from dbd6b87 to 7dfa252 Compare April 28, 2021 14:37
@github-actions github-actions bot temporarily deployed to branch-gh-1753-luajit-profiler April 28, 2021 14:38 Inactive
Copy link
Contributor

@igormunkin igormunkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@veod32, thanks for the fixes! Everything is awesome! LGTM.

There are three more comments below, but they are not critical, so resolve them as you see fit.

Copy link

@Buristan Buristan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fixes, LGTM!

@veod32 veod32 marked this pull request as ready for review April 29, 2021 15:31
@veod32 veod32 merged commit f51577f into latest Apr 29, 2021
@veod32 veod32 deleted the veod32/gh-1753-luajit-profiler branch April 29, 2021 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[8pt] Describe LuaJIT memory profiler interface and parser
3 participants