{{ message }}
Bump to 0.2.9#5
Merged
Merged
Conversation
Get test suite green again, via minitest
If you are using the same reporter instance, after multiple file search the files list will be empty
Don`t modify memoized files list
After this change you can dump your profile results to any IO via:
report = StackProf::Report.new(StackProf.results)
File.open("profile.dump", "wb") do |io|
report.print_dump(io)
end
Pass optional io to Report#print_dump
Fix filtering for Graphviz report
Add option to drop less important nodes for big graphviz output
…iddleware Add raw option for StackProf::Middleware
middleware: pass raw to start
Create LICENSE
Use #print_method instead of #print_source in README after renaming
Documentation
I had to dive into the C source to figure out how to call StackProf without a block because the methods listed in the README don't show their arguments.
Better method examples
`__LINE__` in test/test_stackprof.rb points profiling targets. This commit clear where we focus in test cases.
Describe where `__LINE__` points
…porting Avoid "stack level too deep" error
…_on_reporting Revert "Avoid "stack level too deep" error"
I'm not sure what commit changed this, but if we don't provide a value for `Data_Wrap_Struct`, the object won't get marked. Since the object doesn't get marked, frames we've collected won't get marked and we'll get a segv at report time. Fixes tmm1#58
prevent frames from being GC'd
Don't escape method filter regexp
…porting Avoid "stack level too deep" error
d5ad225 to
a642f06
Compare
Member
a642f06 to
435df56
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

@fbogsany
this is just a bump to 0.2.9. The single change here that affects us is 435df56
which basically just allows us to pass env into the proc call for enabled.
This would correspond with the shopify change for this in https://github.com/Shopify/shopify/pull/72394