Prepend must set method's "defined class" to the new bottom. #4250
Conversation
|
Note that I could not find any code in MRI that does this method-table-walking, so I'm confused how they realign methods to the new hierarchy. This appears to pass specs, but travis is a bit red right now. |
|
I'm going to merge this. The method-walking already was occurring and this just adds an additional bit of state. I believe MRI does not need to walk the method table because their method structs do not contain the originating class. Instead, when they search the method table, they produce a separate struct specific to that method table and its host class. Basically, they lazily determine what we store as implClass based on where the method is actually found at runtime. We may want to adopt this approach in the future. I believe this would help fix the problems we have with method table cloning + super going up the wrong hierarchy. cc @enebo @subbuss. |
This is to see whether jruby/jruby#4250 has solved the problem in #274.
## 1.3.0 * Ensure all tests run individually - thanks to @chrisroos (#267) * Update Travis CI build status badge to show master branch status (#264) * Correct RSpec section of the README - thanks to @myronmarston (0cc039c8) * Fix pretty printing of quotes in `String#mocha_inspect` (#215 & #223) * Add release instructions to README - thanks to @chrisroos (70a5febd & 3c664df7) * Require at least Ruby v1.8.7 in gemspec - thanks to @knappe (3e20be8e) * Remove redundant InstanceMethod#method_exists? - thanks to @chrisroos (8f58eddf) * Reduce risk of hitting bug 12832 in Ruby v2.3 - thanks to @chrisroos (#277 & eca7560c) * Fix JRuby build - thanks to @headius (jruby/jruby#4250) & @chrisroos (#274) * Add latest stable version of JRuby to Travis CI build matrix (#288) * Fix Ruby v1.8.7 builds on Travis CI (928b5a40 & 460dce5b) * Deprecate passing block to mock object constructor (#290) * Add a known issue to README for Ruby bug 12876 (#276) * Add Ruby 2.4 and ruby-head to Travis CI build matrix - thanks to @junaruga (#297) * Fix `Mocha::ParameterMatchers#includes` for `Array` values - thanks to @timcraft (#302) * Use faster container-based virtual environments for Travis CI builds (#305) * Rename `Mocha::ParameterMatchers::QueryStringMatches` to `QueryString` (#306) * Handle blank parameter value for query string matcher - thanks to @weynsee (#303 & #304) * Rename `Mocha::ParameterMatchers::QueryString` -> `EquivalentUri` (#307) * Use `do ... end` instead of `{ ... }` in acceptance tests - thanks to @chrisroos (#294)
See freerange/mocha#274.