Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Boolean support #14

Open
dagolden opened this issue Oct 25, 2013 · 2 comments
Open

Boolean support #14

dagolden opened this issue Oct 25, 2013 · 2 comments
Labels
Milestone

Comments

@dagolden
Copy link

Ingy has mentioned that the boolean rules are changing and all the special case logic could probably go away.

There is also this ticket complaining about a lack of support reading a boolean: https://rt.cpan.org/Ticket/Display.html?id=58782

We need to resolve how booleans are to be handled for 2.00.

@dagolden
Copy link
Author

ping. any ideas, particularly about the bug reading a boolean?

@rjbs
Copy link
Member

rjbs commented Nov 28, 2013

It would be nice to handle literal /\A(?:null|true|false)\z/i properly. True and false could be interpreted as 1 and '' sanely. I even think the current behavior is livable for now, although lousy. And since I think it should work, it would be good to warn on those literals now. And once you identify them to warn, that's 90% of the work to load them, I'd think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant