Bump to 0.2.9 by shivnagarajan · Pull Request #5 · Shopify/stackprof · GitHub
Skip to content

Bump to 0.2.9#5

Merged
shivnagarajan merged 46 commits into
masterfrom
bump_to_0.2.9
May 17, 2016
Merged

Bump to 0.2.9#5
shivnagarajan merged 46 commits into
masterfrom
bump_to_0.2.9

Conversation

@shivnagarajan

@shivnagarajan shivnagarajan commented May 17, 2016

Copy link
Copy Markdown

@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

csfrancis and others added 30 commits February 12, 2014 21:37
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
Use #print_method instead of #print_source in README after renaming
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.
tmm1 and others added 15 commits December 15, 2015 09:31
`__LINE__` in test/test_stackprof.rb points profiling targets.
This commit clear where we focus in test cases.
…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
…porting

Avoid "stack level too deep" error
@shivnagarajan shivnagarajan force-pushed the bump_to_0.2.9 branch 3 times, most recently from d5ad225 to a642f06 Compare May 17, 2016 17:53
@fbogsany

Copy link
Copy Markdown
Member

@shivnagarajan shivnagarajan merged commit a905756 into master May 17, 2016
@shivnagarajan shivnagarajan deleted the bump_to_0.2.9 branch May 17, 2016 18:40
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.