Ticket #26 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

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

Reported by: zbynek.winkler@… Owned by: 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

Changed 5 years ago 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))

Author


E-mail address and user name can be saved in the Preferences.


Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.