Auto merge of #90883 - matthiaskrgr:rollup-iu9k5pe, r=matthiaskrgr
Rollup of 3 pull requests Successful merges: - #90771 (Fix trait object error code) - #90840 (relate lifetime in `TypeOutlives` bounds on drop impls) - #90853 (rustdoc: Use an empty Vec instead of Option<Vec>) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
This commit is contained in:
commit
589ad6a5b7
7 changed files with 46 additions and 18 deletions
|
@ -240,8 +240,12 @@ fn ensure_drop_predicates_are_implied_by_item_defn<'tcx>(
|
||||||
ty::PredicateKind::ConstEvaluatable(a),
|
ty::PredicateKind::ConstEvaluatable(a),
|
||||||
ty::PredicateKind::ConstEvaluatable(b),
|
ty::PredicateKind::ConstEvaluatable(b),
|
||||||
) => tcx.try_unify_abstract_consts((a, b)),
|
) => tcx.try_unify_abstract_consts((a, b)),
|
||||||
(ty::PredicateKind::TypeOutlives(a), ty::PredicateKind::TypeOutlives(b)) => {
|
(
|
||||||
relator.relate(predicate.rebind(a.0), p.rebind(b.0)).is_ok()
|
ty::PredicateKind::TypeOutlives(ty::OutlivesPredicate(ty_a, lt_a)),
|
||||||
|
ty::PredicateKind::TypeOutlives(ty::OutlivesPredicate(ty_b, lt_b)),
|
||||||
|
) => {
|
||||||
|
relator.relate(predicate.rebind(ty_a), p.rebind(ty_b)).is_ok()
|
||||||
|
&& relator.relate(predicate.rebind(lt_a), p.rebind(lt_b)).is_ok()
|
||||||
}
|
}
|
||||||
_ => predicate == p,
|
_ => predicate == p,
|
||||||
}
|
}
|
||||||
|
|
|
@ -952,7 +952,7 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
|
||||||
let mut err = rustc_errors::struct_span_err!(
|
let mut err = rustc_errors::struct_span_err!(
|
||||||
self.sess(),
|
self.sess(),
|
||||||
self_ty.span,
|
self_ty.span,
|
||||||
E0783,
|
E0782,
|
||||||
"{}",
|
"{}",
|
||||||
msg,
|
msg,
|
||||||
);
|
);
|
||||||
|
|
|
@ -205,7 +205,6 @@ crate fn get_index_search_type<'tcx>(
|
||||||
|
|
||||||
inputs.retain(|a| a.ty.name.is_some());
|
inputs.retain(|a| a.ty.name.is_some());
|
||||||
output.retain(|a| a.ty.name.is_some());
|
output.retain(|a| a.ty.name.is_some());
|
||||||
let output = if output.is_empty() { None } else { Some(output) };
|
|
||||||
|
|
||||||
Some(IndexItemFunctionType { inputs, output })
|
Some(IndexItemFunctionType { inputs, output })
|
||||||
}
|
}
|
||||||
|
|
|
@ -117,7 +117,7 @@ crate struct RenderType {
|
||||||
#[derive(Debug)]
|
#[derive(Debug)]
|
||||||
crate struct IndexItemFunctionType {
|
crate struct IndexItemFunctionType {
|
||||||
inputs: Vec<TypeWithKind>,
|
inputs: Vec<TypeWithKind>,
|
||||||
output: Option<Vec<TypeWithKind>>,
|
output: Vec<TypeWithKind>,
|
||||||
}
|
}
|
||||||
|
|
||||||
impl Serialize for IndexItemFunctionType {
|
impl Serialize for IndexItemFunctionType {
|
||||||
|
@ -126,21 +126,16 @@ impl Serialize for IndexItemFunctionType {
|
||||||
S: Serializer,
|
S: Serializer,
|
||||||
{
|
{
|
||||||
// If we couldn't figure out a type, just write `null`.
|
// If we couldn't figure out a type, just write `null`.
|
||||||
let mut iter = self.inputs.iter();
|
let has_missing = self.inputs.iter().chain(self.output.iter()).any(|i| i.ty.name.is_none());
|
||||||
if match self.output {
|
if has_missing {
|
||||||
Some(ref output) => iter.chain(output.iter()).any(|i| i.ty.name.is_none()),
|
|
||||||
None => iter.any(|i| i.ty.name.is_none()),
|
|
||||||
} {
|
|
||||||
serializer.serialize_none()
|
serializer.serialize_none()
|
||||||
} else {
|
} else {
|
||||||
let mut seq = serializer.serialize_seq(None)?;
|
let mut seq = serializer.serialize_seq(None)?;
|
||||||
seq.serialize_element(&self.inputs)?;
|
seq.serialize_element(&self.inputs)?;
|
||||||
if let Some(output) = &self.output {
|
match self.output.as_slice() {
|
||||||
if output.len() > 1 {
|
[] => {}
|
||||||
seq.serialize_element(&output)?;
|
[one] => seq.serialize_element(one)?,
|
||||||
} else {
|
all => seq.serialize_element(all)?,
|
||||||
seq.serialize_element(&output[0])?;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
seq.end()
|
seq.end()
|
||||||
}
|
}
|
||||||
|
|
13
src/test/ui/dropck/relate_lt_in_type_outlives_bound.rs
Normal file
13
src/test/ui/dropck/relate_lt_in_type_outlives_bound.rs
Normal file
|
@ -0,0 +1,13 @@
|
||||||
|
struct Wrapper<'a, T>(&'a T)
|
||||||
|
where
|
||||||
|
T: 'a;
|
||||||
|
|
||||||
|
impl<'a, T> Drop for Wrapper<'a, T>
|
||||||
|
where
|
||||||
|
T: 'static,
|
||||||
|
//~^ error: `Drop` impl requires `T: 'static` but the struct it is implemented for does not
|
||||||
|
{
|
||||||
|
fn drop(&mut self) {}
|
||||||
|
}
|
||||||
|
|
||||||
|
fn main() {}
|
17
src/test/ui/dropck/relate_lt_in_type_outlives_bound.stderr
Normal file
17
src/test/ui/dropck/relate_lt_in_type_outlives_bound.stderr
Normal file
|
@ -0,0 +1,17 @@
|
||||||
|
error[E0367]: `Drop` impl requires `T: 'static` but the struct it is implemented for does not
|
||||||
|
--> $DIR/relate_lt_in_type_outlives_bound.rs:7:8
|
||||||
|
|
|
||||||
|
LL | T: 'static,
|
||||||
|
| ^^^^^^^
|
||||||
|
|
|
||||||
|
note: the implementor must specify the same requirement
|
||||||
|
--> $DIR/relate_lt_in_type_outlives_bound.rs:1:1
|
||||||
|
|
|
||||||
|
LL | / struct Wrapper<'a, T>(&'a T)
|
||||||
|
LL | | where
|
||||||
|
LL | | T: 'a;
|
||||||
|
| |__________^
|
||||||
|
|
||||||
|
error: aborting due to previous error
|
||||||
|
|
||||||
|
For more information about this error, try `rustc --explain E0367`.
|
|
@ -1,4 +1,4 @@
|
||||||
error[E0783]: trait objects without an explicit `dyn` are deprecated
|
error[E0782]: trait objects without an explicit `dyn` are deprecated
|
||||||
--> $DIR/dyn-trait-sugg-2021.rs:10:5
|
--> $DIR/dyn-trait-sugg-2021.rs:10:5
|
||||||
|
|
|
|
||||||
LL | Foo::hi(123);
|
LL | Foo::hi(123);
|
||||||
|
@ -6,4 +6,4 @@ LL | Foo::hi(123);
|
||||||
|
|
||||||
error: aborting due to previous error
|
error: aborting due to previous error
|
||||||
|
|
||||||
For more information about this error, try `rustc --explain E0783`.
|
For more information about this error, try `rustc --explain E0782`.
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue