Modify

Ticket #26 (closed defect: fixed)

Opened 8 years ago

Last modified 8 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

comment:1 Changed 8 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 :(

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


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

 
Note: See TracTickets for help on using tickets.