Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: jruby/jruby
base: eed14f749028
Choose a base ref
...
head repository: jruby/jruby
compare: db484cc30a54
Choose a head ref
  • 3 commits
  • 3 files changed
  • 2 contributors

Commits on Apr 2, 2015

  1. Support respond_to_missing? in respondsTo() lookups 

    RubyBasicObject assumes that if respond_to? is undefined, it can just fast-track
method lookups. We have to also check for respond_to_missing?; failure to do so causes
Javaland respondsTo() calls to fail when an object defines respond_to_missing? (despite
the documentation's admonishment to not use it)

The practical case for this is ActiveSupport::Duration in ActiveSupport 4.2.1
    @cheald
    cheald committed Apr 2, 2015
    Configuration menu
    Copy the full SHA
    fe8c87a View commit details
    Browse the repository at this point in the history

Commits on Apr 4, 2015

  1. Don't depend on method.getName() to have the right name for respond_t… 

    …o/respond_to_missing
    @cheald
    cheald committed Apr 4, 2015
    Configuration menu
    Copy the full SHA
    7708f4c View commit details
    Browse the repository at this point in the history

Commits on Apr 6, 2015

  1. Merge pull request #2797 from cheald/fix-respond-to-missing 

    Support respond_to_missing? in respondsTo() lookups
    @enebo
    enebo committed Apr 6, 2015
    Configuration menu
    Copy the full SHA
    db484cc View commit details
    Browse the repository at this point in the history