-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
RFC: Name visibility determined by capitalization of first letter. #8765
Comments
The naming conventions of It's possible that the |
I'd prefer not to do this, I think we've discussed the possibility before and rejected it, happy to ask the others to confirm. |
@graydon My bad then, I asked on IRC and they told me nobody proposed this. |
triage, as @thestinger says this would be a fundamental change to the naming conventions of the language, and so is unlikely to happen for that reason (and since this has been open 4 months without any activity, I'm going to close). (Personally I think that capitals-for-visibility end up encoding an enforced naming inconsistency into the language, and looks kinda weird too.) cc #8122 for the removal of |
I know 0.8 is "almost there" and from what I understand Rust is now closer to removing features than adding new ones but it's not 1.0 yet so better to contribute.
I come from Go and after almost a year of using it, the best readability feature (aside from gofmt [*]) it has is not having public and private keywords, instead just looking how something is written you already know it, just by reading it without checking docs, wherever they are.
I'm moving to Rust and that's what I miss.
As always it costs something:
EDIT: Just as sidenote, I forgot to mention that this should support Unicode capitalisation which isn't trivial but it's still worth it and there's the guarantee that it only happens at compile time, one character per identifier.
[_]: _gofmt* is awesome and hopefully there's some folks working on rustfmt but it's not ready.
The text was updated successfully, but these errors were encountered: