Skip to content
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

Rollup of PRs in the queue; Friday #23548

Merged
merged 28 commits into from
Mar 20, 2015
Merged

Rollup of PRs in the queue; Friday #23548

merged 28 commits into from
Mar 20, 2015

Conversation

Manishearth
Copy link
Member

No description provided.

kjpgit and others added 15 commits March 17, 2015 11:52
These two borrowing examples were confusing/misleading.  This changes it
to more clearly show how you _can_ borrow a box, and also uses & instead
of &*.
Multiple people have been suprised by this aspect of read_line's behavior,
which is not obvious from the docs.
This fixes several use cases that were broken after rust-lang#23265 landed.
We don't use 'task' anymore, these are now threads.

Because this changes the name of a compiler option, this is

[breaking-change]
`uppercase` and `lowercase` are currently named `to_uppercase` and `to_lowercase`.
Also adds a link to the `char` type documentation which has much more detail
on these iterators.
…erflow-bug, r=alexcrichton

 This should solve issues rust-lang#23115, rust-lang#23469, and rust-lang#23407.

As the title says, this is just a workaround. The underlying problem is that macro expansion can produce invalid spans. I've opened issue rust-lang#23480 so we don't forget about that.
As @alexcrichton says, this was really a libgreen thing, and isn't
relevant now.

As this removes a technically-public function, this is a

[breaking-change]

Conflicts:
	src/libtest/lib.rs
@Manishearth
Copy link
Member Author

@bors: r+ p=20

@bors
Copy link
Contributor

bors commented Mar 20, 2015

📌 Commit 4732e7f has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Mar 20, 2015

⌛ Testing commit 4732e7f with merge ab566e0...

@bors
Copy link
Contributor

bors commented Mar 20, 2015

💔 Test failed - auto-win-32-opt

@Manishearth
Copy link
Member Author

@bors: r+ force

@bors
Copy link
Contributor

bors commented Mar 20, 2015

📌 Commit 6107e4c has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Mar 20, 2015

⌛ Testing commit 6107e4c with merge 0834bd1...

…r=aturon

 The IP and socket address types all had `FromStr` implemented but the
implementations were not marked stable, nor was the error type returned ready to
be properly stabilized.

This commit marks the implementations of `FromStr` as stable and also renamed
the `ParseError` structure to `AddrParseError`. The error is now also an opaque
structure that cannot be constructed outside the standard library.

cc rust-lang#22949
[breaking-change]
…hton

 We don't use 'task' anymore, these are now threads.

Because this changes the name of a compiler option, this is

[breaking-change]

I think this is small enough to not need an RFC, nor a period of accepting both. If we want to take both for a while, I can change the patch.
…labnik

 These two borrowing examples were confusing/misleading.  This changes it
to more clearly show how you _can_ borrow a box, and also uses & instead
of &*.
 Multiple people have been suprised by this aspect of read_line's behavior, which is not obvious from the docs.
 This fixes several use cases that were broken after rust-lang#23265 landed.
 `uppercase` and `lowercase` are currently named `to_uppercase` and `to_lowercase`. Also adds a link to the `char` type documentation which has much more detail on these iterators.
 Fixes rust-lang#16330

My troff is probably poor, so make sure that all looks okay.
…r=alexcrichton

As @alexcrichton says, this was really a libgreen thing, and isn't
relevant now.

As this removes a technically-public function, this is a

[breaking-change]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants