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

deprecate leading 0 for octal #5205

Closed
scabug opened this issue Nov 18, 2011 · 7 comments
Closed

deprecate leading 0 for octal #5205

scabug opened this issue Nov 18, 2011 · 7 comments
Assignees

Comments

@scabug
Copy link

@scabug scabug commented Nov 18, 2011

Community consensus seems to be overwhelmingly in favor of deprecating 012 == 10, i.e. leading-0-means-octal. I propose to follow through on it in the absence of objections.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Nov 18, 2011

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Nov 29, 2011

@gkossakowski said:
Isn't it already done in trunk?

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Nov 29, 2011

@soc said:
That was only the “double literal without number after the dot” (is there a less bulky name?) afaik.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Nov 29, 2011

Commit Message Bot (anonymous) said:
(extempore in [r26091|https://codereview.scala-lang.org/fisheye/changelog/scala-svn?cs=26091]) Fixed -Xfuture 5.ds, deprecated 0-octal.

I messed up my trip to the future the first time around; now in
the future 5.f is not an error but an attempt to call method "f"
on 5 like nature intended. (Thank you simon for catching this.)
And deprecated leading 0 for octal. Closes #5205.

@scabug scabug closed this Nov 29, 2011
@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Nov 29, 2011

@ijuma said:
Yay! :)

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Nov 18, 2012

@soc said:
Spec update is scala/scala-dist#20.

@scabug

This comment has been minimized.

Copy link
Author

@scabug scabug commented Nov 25, 2014

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

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.