Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Testcases for some fixed bugs #8209

Closed

Conversation

graydon
Copy link
Contributor

@graydon graydon commented Aug 2, 2013

This just fills in a few bugs that needed tests.

r? @alexcrichton

// option. This file may not be copied, modified, or distributed
// except according to those terms.

fn make_adder(x: int) -> @fn(int) -> int { |y| x + y }
Copy link
Member

Choose a reason for hiding this comment

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

Doesn't this test need a fn main to pass?

@alexcrichton
Copy link
Member

Nice tests!

@graydon
Copy link
Contributor Author

graydon commented Aug 15, 2013

continued into rollup, closing this PR.

@graydon graydon closed this Aug 15, 2013
bors added a commit that referenced this pull request Aug 20, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 13, 2022
…uct, r=llogiq

return_self_not_must_use document `#[must_use]` on the type

Inspired by a discussion in rust-lang/rust-clippy#8197

---

r? `@llogiq`

changelog: none

The lint is this on nightly, therefore no changelog entry for you xD
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants