Warn about bug #4044 in documentation until it gets fixed#443
Closed
rosenfeld wants to merge 1 commit intoruby:trunkfrom
Closed
Warn about bug #4044 in documentation until it gets fixed#443rosenfeld wants to merge 1 commit intoruby:trunkfrom
rosenfeld wants to merge 1 commit intoruby:trunkfrom
Conversation
The bug has been reported almost 3 years ago, so we may believe it could stand open for more time. Until it's fixed, users should be aware of it when willing to use the \W character class.
Author
|
Thanks :) |
mmasaki
pushed a commit
to mmasaki/ruby
that referenced
this pull request
Nov 13, 2013
duerst-san in [ruby-core:43612] [Fixes rubyGH-443] Patched by @rosenfeld ruby#443 git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@43657 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
tenderlove
pushed a commit
to tenderlove/ruby
that referenced
this pull request
Jan 24, 2014
duerst-san in [ruby-core:43612] [Fixes rubyGH-443] Patched by @rosenfeld ruby#443 git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@43657 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
hsbt
pushed a commit
that referenced
this pull request
Sep 25, 2020
* Enhanced RDoc for JSON.dump ruby/json@03f1699ec4
ioquatix
pushed a commit
to ioquatix/ruby
that referenced
this pull request
Sep 30, 2020
* Enhanced RDoc for JSON.dump ruby/json@03f1699ec4
matzbot
pushed a commit
that referenced
this pull request
Aug 31, 2021
'y' and 'n' are kind of ambiguous. Syck treated y and n literals in YAML documents as strings. But this is not what the YAML 1.1 spec says. YAML 1.1 says they should be treated as booleans. When we're dumping documents, we know it's a string, so adding quotes will eliminate the "ambiguity" in the emitted document Fixes #443 ruby/psych@6a1c30634e
maximecb
added a commit
that referenced
this pull request
Aug 31, 2022
* Better b.cond usage on AArch64 When we're lowering a conditional jump, we previously had a bit of a complicated setup where we could emit a conditional jump to skip over a jump that was the next instruction, and then write out the destination and use a branch register. Now instead we use the b.cond instruction if our offset fits (not common, but not unused either) and if it doesn't we write out an inverse condition to jump past loading the destination and branching directly. * Added an inverse fn for Condition (#443) Prevents the need to pass two params and potentially reduces errors. Co-authored-by: Jimmy Miller <jimmyhmiller@jimmys-mbp.lan> Co-authored-by: Maxime Chevalier-Boisvert <maximechevalierb@gmail.com> Co-authored-by: Jimmy Miller <jimmyhmiller@jimmys-mbp.lan>
matzbot
pushed a commit
that referenced
this pull request
Jan 17, 2024
matzbot
pushed a commit
that referenced
this pull request
Jan 17, 2024
… covered in the more general test ruby/psych@6750b35402
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The bug has been reported almost 3 years ago, so we may believe it could stand open for more time. Until it's fixed, users should be aware of it when willing to use the \W character class.