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
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 55fa7734af83
Choose a base ref
...
head repository: jruby/jruby
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 652d8a2a134c
Choose a head ref
  • 2 commits
  • 2 files changed
  • 2 contributors

Commits on Mar 21, 2018

  1. Freeze Complex and Rational

    Complex and Rational are always frozen now.
    See ruby/ruby@42c9819 in MRI.
    ChrisBr committed Mar 21, 2018
    Copy the full SHA
    a52f80e View commit details
  2. Merge pull request #5102 from ChrisBr/freeze-complex

    Freeze Complex and Rational
    headius authored Mar 21, 2018
    Copy the full SHA
    652d8a2 View commit details
Showing with 2 additions and 0 deletions.
  1. +1 −0 core/src/main/java/org/jruby/RubyComplex.java
  2. +1 −0 core/src/main/java/org/jruby/RubyRational.java
1 change: 1 addition & 0 deletions core/src/main/java/org/jruby/RubyComplex.java
Original file line number Diff line number Diff line change
@@ -125,6 +125,7 @@ private RubyComplex(Ruby runtime, RubyClass clazz, IRubyObject real, IRubyObject
super(runtime, clazz);
this.real = real;
this.image = image;
this.flags |= FROZEN_F;
}

/** rb_complex_raw
1 change: 1 addition & 0 deletions core/src/main/java/org/jruby/RubyRational.java
Original file line number Diff line number Diff line change
@@ -90,6 +90,7 @@ private RubyRational(Ruby runtime, RubyClass clazz, RubyInteger num, RubyInteger
super(runtime, clazz);
this.num = num;
this.den = den;
this.flags |= FROZEN_F;
}

private RubyRational(Ruby runtime, RubyClass clazz, IRubyObject num, IRubyObject den) {