-
-
Notifications
You must be signed in to change notification settings - Fork 925
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
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: e8ef7937dbca
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: jruby/jruby
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: a35d850f5184
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 9 commits
- 16 files changed
- 3 contributors
Commits on Feb 23, 2018
-
Fix regressions after chr fixes. uptoCommon was changing our single byte
encodings because they were shared String buts setEncoding was changing their encoding. This seems like a tiny fix but it has a large change in it. If we setEncoding() on a shared String it will actually stop sharing the underlying bytelist. In practice, this could mean some extra bytelist creation behind the scenes. What it fixes is n strings sharing the same bytelist suddenly getting a new encoding because one decided to have force_encoding called on it. Making that tiny change also forced a couple of other changes because flags for sharing are fragile due to order dependency when constructing Ruby Strings. Also tweaked ordering of a conditional which wrankled my delicate sensibilities.
Configuration menu - View commit details
-
Copy full SHA for 69998e8 - Browse repository at this point
Copy the full SHA 69998e8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5c502e3 - Browse repository at this point
Copy the full SHA 5c502e3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4a3214c - Browse repository at this point
Copy the full SHA 4a3214cView commit details -
Fixes #5052. String literal difference from MRI.
Basic gist of fix is to push '#' onto result buffer if we did not find any useful characters after the next significant character ('@', '$'). Fixing this was confusing because we have an ancient outdated pushback(char) method which actually ignores c (except for eof).
Configuration menu - View commit details
-
Copy full SHA for 27b8700 - Browse repository at this point
Copy the full SHA 27b8700View commit details -
Added specs for partial string interpolation sequences which are not …
…actual interpolations. They should just reflect the original string.
Configuration menu - View commit details
-
Copy full SHA for ad3ecd3 - Browse repository at this point
Copy the full SHA ad3ecd3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9b68af9 - Browse repository at this point
Copy the full SHA 9b68af9View commit details -
Configuration menu - View commit details
-
Copy full SHA for bbb8682 - Browse repository at this point
Copy the full SHA bbb8682View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2a1d9a5 - Browse repository at this point
Copy the full SHA 2a1d9a5View commit details -
I feel like I am in some git nightmare...Redid the merge I already di…
…d to have another merge undo it...
Configuration menu - View commit details
-
Copy full SHA for a35d850 - Browse repository at this point
Copy the full SHA a35d850View commit details
Loading
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff e8ef7937dbca...a35d850f5184