Actions
Bug #7636
closedProblems in the documnt of psych
Description
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.
Files
Updated by zzak (zzak _) almost 12 years ago
- Assignee set to zzak (zzak _)
- Target version set to 2.0.0
I'll have to check this upstream, psych is maintained on github (https://github.com/tenderlove/psych), but documentation patches here are fine.
Updated by zzak (zzak _) almost 12 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]
Actions
Like0
Like0Like0