Skip to content
Snippets Groups Projects

Update allocations Gem & ignore classes without names

Merged yorickpeterse-staging requested to merge update-allocations-gem-fix-names into master

What does this MR do?

  1. Updates the allocations Gem to 1.0.5 so it can be used on 2.2/2.3 without crashing
  2. Changes the background sampler to ignore classes/modules without a name (as we can't do much with these)

Are there points in the code the reviewer needs to double check?

No.

Why was this MR needed?

Using MRI 2.2 or 2.3 would lead to a segmentation fault in the allocations Gem, sometimes this problem would also occur on 2.1

What are the relevant issue numbers?

Not an issue, but the merge request that highlighted these problems: !3807 (merged)

Does this MR meet the acceptance criteria?

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading