Skip to content

Clean up some FIXMEs.#37104

Merged
bors merged 4 commits into
rust-lang:masterfrom
luqmana:fixmes
Nov 11, 2016
Merged

Clean up some FIXMEs.#37104
bors merged 4 commits into
rust-lang:masterfrom
luqmana:fixmes

Conversation

@luqmana

@luqmana luqmana commented Oct 12, 2016

Copy link
Copy Markdown
Contributor

Remove some assorted FIXMEs in the codebase.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @pnkfelix

(rust_highfive has picked a reviewer for you, use r? to override)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

does this represent a change in analysis behavior?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not that I could see and passed make check. Also @nikomatsakis mentioned on the issue that he didn't expect any effect.

@alexcrichton

Copy link
Copy Markdown
Member

ping r? @pnkfelix

@bors

bors commented Nov 9, 2016

Copy link
Copy Markdown
Collaborator

☔ The latest upstream changes (presumably #37670) made this pull request unmergeable. Please resolve the merge conflicts.

@alexcrichton

Copy link
Copy Markdown
Member

@eddyb, thoughts here? (looks fairly trivial)

@luqmana want to rebase?

@alexcrichton alexcrichton added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Nov 10, 2016
@eddyb

eddyb commented Nov 10, 2016

Copy link
Copy Markdown
Contributor

r=me w/ rebase.

@luqmana

luqmana commented Nov 11, 2016

Copy link
Copy Markdown
Contributor Author

@bors r=eddyb

@eddyb

eddyb commented Nov 11, 2016

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Nov 11, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit c2f1e5d has been approved by eddyb

@bors

bors commented Nov 11, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit c2f1e5d with merge de46bea...

bors added a commit that referenced this pull request Nov 11, 2016
Clean up some FIXMEs.

Remove some assorted FIXMEs in the codebase.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants