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:


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 one would expect it to be there.


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:, where x,y are used to specify a point. It's just that nobody bothered to update the spec :(


Add a comment

Modify Ticket

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

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

Note: See TracTickets for help on using tickets.