dox4/ruby
1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2026-08-16 02:15:57 +08:00
Code Issues Packages Projects Releases Wiki Activity

[ruby/psych] Resolve booleans per YAML 1.2 on the libfyaml backend

Browse Source 
Scalar type resolution happens in ScalarScanner, not the C backend, so
swapping to libfyaml alone still resolved yes/no/on/off to booleans. Key
the boolean set on Psych::BACKEND so the libyaml default keeps the YAML
1.1 set while the experimental libfyaml backend follows 1.2.

https://github.com/ruby/psych/commit/b374869c59

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Hiroshi SHIBATA 2026-07-01 07:00:30 +09:00 committed by git
parent ce089a3de1
commit 4e225df5ed
1 changed files with 14 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
ext/psych/lib/psych/scalar_scanner.rb
View File

@ -24,6 +24,18 @@ module Psych
|[-+]?(?:0|[1-9](?:[0-9]|,[0-9]|_[0-9])*) (?# base 10)
|[-+]?0x[_,]*[0-9a-fA-F][0-9a-fA-F_,]* (?# base 16))$/x
# YAML 1.1 treats yes/no/on/off as booleans in addition to true/false,
# while YAML 1.2's core schema only recognizes true/false. The default
# libyaml backend keeps the 1.1 set for backward compatibility; the
# experimental libfyaml backend follows 1.2.
if defined?(Psych::BACKEND) && Psych::BACKEND == 'libfyaml'
BOOLEAN_TRUE = /^true$/i
BOOLEAN_FALSE = /^false$/i
else
BOOLEAN_TRUE = /^(yes|true|on)$/i
BOOLEAN_FALSE = /^(no|false|off)$/i
end
attr_reader :class_loader
# Create a new scanner
@ -48,9 +60,9 @@ module Psych
string
elsif string == '~' || string.match?(/^null$/i)
nil
elsif string.match?(/^(yes|true|on)$/i)
elsif string.match?(BOOLEAN_TRUE)
true
elsif string.match?(/^(no|false|off)$/i)
elsif string.match?(BOOLEAN_FALSE)
false
else
string