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

Add ⏨ as exponent notation for floats. #1348

Closed
wants to merge 1 commit into from
Closed

Conversation

clsn
Copy link
Contributor

@clsn clsn commented Dec 29, 2017

To address #1347 in case anyone eventually cares.

@zoffixznet
Copy link
Contributor

👎 from me on this.

I doubt many people will use this and it's even in the same token that has the U+2212 minus, which in the past caused tons of bugs, that is still buggy in cases, and that I've never seen anyone use.

This falls squarely into "you think this is cute now" category that we'd be regretting in the future.

@zoffixznet
Copy link
Contributor

which in the past caused tons of bugs

Already spotted one bug were this PR accepted: &val (and subsequently Str.Numeric) won't know how to handle this syntax and would need fixing. I suspect sprintf won't either and that lives in nqp.

@AlexDaniel
Copy link
Contributor

AlexDaniel commented Dec 29, 2017

Yeah we'll have to put that into the drawer for now. Who knows, maybe we'll get back to it later. In any case, it's now listed on the wiki and I even added a link to this PR. Further discussion is welcome but this won't be added in the very near future.

@AlexDaniel AlexDaniel closed this Dec 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants