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 PRs in the queue #21997

Merged
merged 58 commits into from
Feb 7, 2015
Merged

Rollup PRs in the queue #21997

merged 58 commits into from
Feb 7, 2015

Conversation

Manishearth
Copy link
Member

No description provided.

steveklabnik and others added 30 commits February 4, 2015 02:19
This is now a Result, not an Option.
This was particularly helpful in the time just after OIBIT's
implementation to make sure things that were supposed to be Copy
continued to be, but it's now creates a lot of noise for types that
intentionally don't want to be Copy.
Currently "k / 2" generates one (k: uint) or two (k: int) "br false,
..." instructions and the corresponding basic blocks, producing quite
some noise and making the code unnecessarily hard to read.

Additionally we can skip translation if the code would end up
unreachable anyway.
The spelling corrections were made in both documentation comments and
regular comments.
Just spelling corrections.
…ult.

The 'stable_features' lint helps people progress from unstable to
stable Rust by telling them when they no longer need a `feature`
attribute because upstream Rust has declared it stable.

This compares to the existing 'unstable_features', which is used
to implement feature staging, and triggers on *any* use
of `#[feature]`.
New functions, slice::from_raw_parts and slice::from_raw_parts_mut,
are added to implement the lifetime convention as agreed in RFC PR rust-lang#556.
The functions slice::from_raw_buf and slice::from_raw_mut_buf are
left deprecated for the time being.
- add `_SC_GETPW_R_SIZE_MAX` constant
- declare `struct passwd`
- convert `load_self` to `current_exe`

Note: OpenBSD don't provide system function to return a valuable Path
for `env::current_exe`. The implementation is currently based on the
value of `argv[0]`, which couldn't be used when executable is called via
PATH.
- add `std::env:consts`
- deprecating `std::os::consts`
- refactoring errno_location()
Only the second reference's mutability is relevant to the mutability
of the returned reference.
…move this overeager code that was pruning out ambig where-clause matches in trait selection. cc rust-lang#21974.
This can be considered partial work on rust-lang#8256.

The main observable change: macro expansion sometimes results in spans
where `lo > hi`; so for now, when we have such a span, do not attempt
to return a snippet result.

(Longer term, we might think about whether we could still present a
snippet for the cases where this arises, e.g. perhaps by showing the
whole macro as the snippet, assuming that is the sole cause of such
spans; or by somehow looking up the closest AST node that holds both
`lo` and `hi`, and showing that.)

As a drive-by, revised the API to return a `Result` rather than an
`Option`, with better information-packed error value that should help
us (and maybe also our users) identify the causes of such problems in
the future.  Ideally the call-sites that really want an actual snippet
would be updated to catch the newly added `Err` case and print
something meaningful about it, but that is not part of this PR.
(I often run `compiletest` by hand by cut-and-pasting from what `make`
runs, but then I need to tweak it (cut out options) and its useful to
be told when I have removed an option that is actually required, such
as `--android-cross-path=path`.)
…holds

cyclic structure (which the Arena API updated for dropck cannot handle).
…ance analysis.

(The code was not using the main feature of Arena in the first place,
since all of the `alloc` calls were on the same type.
Creating two identical static expressions involving casts of pointers to arrays
caused an assertion failure in librustc_trans.
…nalysis, r=huonw

 Use a TypedArena rather than an Arena for the backing storage of variance analysis.

(The code was not using the main feature of Arena in the first place, since all of the `alloc` calls were on the same type.)
 Creating two identical static expressions involving casts of pointers to arrays
caused an assertion failure in librustc_trans.

cc @eddyb
@pnkfelix
Copy link
Member

pnkfelix commented Feb 6, 2015

@bors r+ 1a2b602 p=23

The word is repeated twice in the message like

    error: obsolete syntax: `:`, `&mut:`, or `&:` syntax

This removes the word syntax that appears in messages after the second colon (:).
@bors
Copy link
Contributor

bors commented Feb 6, 2015

⌛ Testing commit 1a2b602 with merge 8471f31...

bors added a commit that referenced this pull request Feb 6, 2015
@bors
Copy link
Contributor

bors commented Feb 6, 2015

💔 Test failed - auto-mac-64-opt

@alexcrichton
Copy link
Member

@bors: r+ e5f2524

@pnkfelix
Copy link
Member

pnkfelix commented Feb 6, 2015

@bors r+ e5f2524 p=23

Gankra and others added 5 commits February 7, 2015 00:31
Conflicts:
	src/libstd/collections/mod.rs
Conflicts:
	src/libstd/collections/mod.rs
 It seems that `lang_feature_stats` was supposed to refer to `language_feature_stats` instead.

r? @brson
 The word is repeated twice in the message like:

    error: obsolete syntax: `:`, `&mut:`, or `&:` syntax

This removes the word syntax that appears in messages after the second colon (:).
 If you were still using `MaybeOwnedVector`, update your code to use `CowVec`.

[breaking-change]

---

We already removed `MaybeOwned` (the string equivalent) long time ago and with a much shorter deprecation period. It's time to let go.
@alexcrichton
Copy link
Member

@bors: r+ df7db97 p=100

@bors
Copy link
Contributor

bors commented Feb 6, 2015

⌛ Testing commit df7db97 with merge b52f898...

@bors
Copy link
Contributor

bors commented Feb 6, 2015

💔 Test failed - auto-win-32-opt

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented Feb 6, 2015

⌛ Testing commit df7db97 with merge d3732a1...

bors added a commit that referenced this pull request Feb 6, 2015
@bors bors merged commit df7db97 into rust-lang:master Feb 7, 2015
@pnkfelix
Copy link
Member

pnkfelix commented Feb 7, 2015

\o/

@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
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.