Bug #7636

Problems in the documnt of psych

Added by ohai (Ippei Obayashi) almost 8 years ago. Updated almost 8 years ago.

Target version:
ruby -v:
ruby 2.0.0dev (2012-12-01 trunk 38126) [x86_64-linux]


I found some mistakes in the comment of psych library.

The following is the comment of Psych.load:
# Example:
# Psych.load("--- a") # => 'a'
# Psych.load("---\n - a\n - b") # => ['a', 'b']
# begin
# Psych.load("--- `", "file.txt")
# rescue Psych::SyntaxError => ex
# ex.file # => 'file.txt'
# ex.message # => "(foo.txt): found character that cannot start any token"
# end
The return value of ex.message is not correct. "foo.txt" should be "file.txt".
We can find the same errros on some other methods.

The attached patch fixes this problem. The patch also update the message
for newer versions.


psych-doc.patch (1.46 KB) psych-doc.patch ohai (Ippei Obayashi), 12/29/2012 11:38 PM

Updated by zzak (Zachary Scott) almost 8 years ago

  • Assignee set to zzak (Zachary Scott)
  • Target version set to 2.0.0

I'll have to check this upstream, psych is maintained on github (, but documentation patches here are fine.


Updated by zzak (Zachary Scott) almost 8 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r38680.
Ippei, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

  • ext/psych/lib/psych.rb (Psych.load): Return value of Psych::SyntaxError.message should be same as example. Patch by Ippei Obayashi [ruby-core:51193] [Bug #7636]

Also available in: Atom PDF