Ticket #26 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

yaml.load("a: n") raises exception (problem with bool)

Reported by: zbynek.winkler@gmail.com Assigned to: xi
Priority: normal Component: pyyaml
Severity: normal Keywords:
Cc:

Description

r59 claims to "Remove y/n from the boolean constants." However 'n|N' is not removed from the corresponding regexp. BTW: why was it removed? Looking at http://yaml.org/type/bool.html one would expect it to be there.

Attachments

Change History

08/16/06 17:13:45 changed by xi

  • status changed from new to closed.
  • resolution set to fixed.

Fixed in [226].

This issue was discussed on the mailing list and there was consensus that using one-character abbreviations is a bad idea and y/n should be removed from the list of boolean constants. As an example, check the !!merge spec: http://yaml.org/type/merge.html, where x,y are used to specify a point. It's just that nobody bothered to update the spec :(

Add/Change #26 (yaml.load("a: n") raises exception (problem with bool))




Change Properties
Action