Skip to content

Conversation

nrc
Copy link
Member

@nrc nrc commented Dec 23, 2015

As discussed here

r? @nikomatsakis or anyone else on the @rust-lang/compiler team

@jonas-schievink
Copy link
Contributor

@nrc
Copy link
Member Author

nrc commented Dec 23, 2015

Whoops, forgot the new commit

@aturon
Copy link
Contributor

aturon commented Dec 29, 2015

cc @rust-lang/compiler

@nrc
Copy link
Member Author

nrc commented Dec 29, 2015

Addressed the doc comment thing.

The problem with the Option thing was that using the StructErr was painful - lots of weird maps and as_mut stuff. Using the dummy thing is much cleaner to use, although a little bit less clear what is going on.

@nrc
Copy link
Member Author

nrc commented Dec 29, 2015

@bors: r=nagisa

@bors
Copy link
Collaborator

bors commented Dec 29, 2015

📌 Commit 6fd8a9c has been approved by nagisa

@nrc
Copy link
Member Author

nrc commented Dec 30, 2015

@bors: r= nagisa

@bors
Copy link
Collaborator

bors commented Dec 30, 2015

📌 Commit 489aa40 has been approved by ``

@nrc
Copy link
Member Author

nrc commented Dec 30, 2015

@bors: r=nagisa

@bors
Copy link
Collaborator

bors commented Dec 30, 2015

📌 Commit 489aa40 has been approved by nagisa

@nrc
Copy link
Member Author

nrc commented Dec 30, 2015

@bors: r=nagisa

@bors
Copy link
Collaborator

bors commented Dec 30, 2015

📌 Commit 04d9729 has been approved by nagisa

@bors
Copy link
Collaborator

bors commented Dec 30, 2015

⌛ Testing commit 04d9729 with merge 176ee34...

bors added a commit that referenced this pull request Dec 30, 2015
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.

6 participants