Skip to content

Switch <> back to [] #148

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.

Sign up for GitHub

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

Closed
thehydroimpulse wants to merge 11 commits into from
Closed

Switch <> back to [] #148

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
e530574
New syntax proposal
thehydroimpulse Jul 1, 2014
69620ab
Unresolved questions
thehydroimpulse Jul 1, 2014
af716da
Cleared things up
thehydroimpulse Jul 1, 2014
4f9b61a
Better points.
thehydroimpulse Jul 1, 2014
3a787ff
Removed some subjectiveness
thehydroimpulse Jul 1, 2014
bbba791
Fixed generalizing statement
thehydroimpulse Jul 1, 2014
d7dd8ea
Removed subjectiveness
thehydroimpulse Jul 1, 2014
ddabf81
Merged two points. Expanded on another one
thehydroimpulse Jul 1, 2014
f330bdb
Removed useless points
thehydroimpulse Jul 1, 2014
c8c3bd2
## to #
thehydroimpulse Jul 1, 2014
b493af0
Improved RFC from comments (Thanks @ChrisMorgan)
thehydroimpulse Jul 1, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
## to #
  • Loading branch information
@thehydroimpulse
thehydroimpulse committed Jul 1, 2014
commit c8c3bd2e96cedba825322062c6d5ea91891c7cd4
2 changes: 1 addition & 1 deletion active/0000-square-brackets.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ fn parse['a, T: Encodable[Encoder['a], IoError]](value: T) {

This is a very easy change to make.
Copy link
Member

@chris-morgan chris-morgan Jul 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That’s… not much of a detailed design. How about saying something like “in type parameter lists, replace < with [ and > with ].” and giving examples of T<U, …> and f::<T, …>()?

Copy link
Author

@thehydroimpulse thehydroimpulse Jul 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

Copy link
Contributor

@liigo liigo Jul 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The


## Downsides
# Downsides

* The syntax is used quite a bit. Automation could potentially do some, if not most of the changes (The tricky part is the ambiguities in the current syntax). However, of the changes we've had in the past, I think this syntax change is a whole lot easier to work with than semantic changes, or more complex syntax changes.

Expand Down