Skip to content

Commit 65c2e51

Browse files
lnicolalnicola
committed
Rename TyKind::Unknown to Error
1 parent d8ee25b commit 65c2e51

File tree

14 files changed

+48
-48
lines changed

14 files changed

+48
-48
lines changed

‎crates/hir/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1876,7 +1876,7 @@ impl Type {
18761876

18771877
fn go(ty: &Ty) -> bool {
18781878
match ty.kind(&Interner) {
1879-
TyKind::Unknown => true,
1879+
TyKind::Error => true,
18801880

18811881
TyKind::Adt(_, substs)
18821882
| TyKind::AssociatedType(_, substs)

‎crates/hir_ty/src/builder.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ impl<D> TyBuilder<D> {
5454
}
5555

5656
pub fn fill_with_unknown(self) -> Self {
57-
self.fill(iter::repeat(TyKind::Unknown.intern(&Interner)))
57+
self.fill(iter::repeat(TyKind::Error.intern(&Interner)))
5858
}
5959

6060
pub fn fill(mut self, filler: impl Iterator<Item = impl CastTo<GenericArg>>) -> Self {

‎crates/hir_ty/src/display.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -476,7 +476,7 @@ impl HirDisplay for Ty {
476476
parameter.assert_ty_ref(&Interner).kind(&Interner),
477477
default_parameters.get(i),
478478
) {
479-
(&TyKind::Unknown, _) | (_, None) => {
479+
(&TyKind::Error, _) | (_, None) => {
480480
default_from = i + 1;
481481
}
482482
(_, Some(default_parameter)) => {
@@ -636,7 +636,7 @@ impl HirDisplay for Ty {
636636
}
637637
};
638638
}
639-
TyKind::Unknown => {
639+
TyKind::Error => {
640640
if f.display_target.is_source_code() {
641641
return Err(HirDisplayError::DisplaySourceCodeError(
642642
DisplaySourceCodeError::UnknownType,

‎crates/hir_ty/src/infer.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ struct InternedStandardTypes {
120120

121121
impl Default for InternedStandardTypes {
122122
fn default() -> Self {
123-
InternedStandardTypes { unknown: TyKind::Unknown.intern(&Interner) }
123+
InternedStandardTypes { unknown: TyKind::Error.intern(&Interner) }
124124
}
125125
}
126126

@@ -247,7 +247,7 @@ impl<'a> InferenceContext<'a> {
247247
table: unify::InferenceTable::new(),
248248
obligations: Vec::default(),
249249
last_obligations_check: None,
250-
return_ty: TyKind::Unknown.intern(&Interner), // set in collect_fn_signature
250+
return_ty: TyKind::Error.intern(&Interner), // set in collect_fn_signature
251251
trait_env: owner
252252
.as_generic_def_id()
253253
.map_or_else(Default::default, |d| db.trait_environment(d)),
@@ -261,7 +261,7 @@ impl<'a> InferenceContext<'a> {
261261
}
262262

263263
fn err_ty(&self) -> Ty {
264-
TyKind::Unknown.intern(&Interner)
264+
TyKind::Error.intern(&Interner)
265265
}
266266

267267
fn resolve_all(mut self) -> InferenceResult {
@@ -326,7 +326,7 @@ impl<'a> InferenceContext<'a> {
326326
/// Replaces Ty::Unknown by a new type var, so we can maybe still infer it.
327327
fn insert_type_vars_shallow(&mut self, ty: Ty) -> Ty {
328328
match ty.kind(&Interner) {
329-
TyKind::Unknown => self.table.new_type_var(),
329+
TyKind::Error => self.table.new_type_var(),
330330
_ => ty,
331331
}
332332
}
@@ -542,7 +542,7 @@ impl<'a> InferenceContext<'a> {
542542
result
543543
} else {
544544
// FIXME diagnostic
545-
(TyKind::Unknown.intern(&Interner), None)
545+
(TyKind::Error.intern(&Interner), None)
546546
}
547547
}
548548

@@ -755,15 +755,15 @@ impl Expectation {
755755
fn none() -> Self {
756756
Expectation {
757757
// FIXME
758-
ty: TyKind::Unknown.intern(&Interner),
758+
ty: TyKind::Error.intern(&Interner),
759759
rvalue_hint: false,
760760
}
761761
}
762762

763763
fn coercion_target(&self) -> Ty {
764764
if self.rvalue_hint {
765765
// FIXME
766-
TyKind::Unknown.intern(&Interner)
766+
TyKind::Error.intern(&Interner)
767767
} else {
768768
self.ty.clone()
769769
}

‎crates/hir_ty/src/infer/path.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ impl<'a> InferenceContext<'a> {
142142
remaining_segments_for_ty,
143143
true,
144144
);
145-
if let TyKind::Unknown = ty.kind(&Interner) {
145+
if let TyKind::Error = ty.kind(&Interner) {
146146
return None;
147147
}
148148

@@ -207,7 +207,7 @@ impl<'a> InferenceContext<'a> {
207207
name: &Name,
208208
id: ExprOrPatId,
209209
) -> Option<(ValueNs, Option<Substitution>)> {
210-
if let TyKind::Unknown = ty.kind(&Interner) {
210+
if let TyKind::Error = ty.kind(&Interner) {
211211
return None;
212212
}
213213

‎crates/hir_ty/src/infer/unify.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ impl TypeVariableTable {
214214
fn fallback_value(&self, iv: InferenceVar, kind: TyVariableKind) -> Ty {
215215
match kind {
216216
_ if self.inner[iv.to_inner().0 as usize].diverging => TyKind::Never,
217-
TyVariableKind::General => TyKind::Unknown,
217+
TyVariableKind::General => TyKind::Error,
218218
TyVariableKind::Integer => TyKind::Scalar(Scalar::Int(IntTy::I32)),
219219
TyVariableKind::Float => TyKind::Scalar(Scalar::Float(FloatTy::F64)),
220220
}
@@ -327,7 +327,7 @@ impl InferenceTable {
327327

328328
pub(super) fn unify_inner_trivial(&mut self, ty1: &Ty, ty2: &Ty, depth: usize) -> bool {
329329
match (ty1.kind(&Interner), ty2.kind(&Interner)) {
330-
(TyKind::Unknown, _) | (_, TyKind::Unknown) => true,
330+
(TyKind::Error, _) | (_, TyKind::Error) => true,
331331

332332
(TyKind::Placeholder(p1), TyKind::Placeholder(p2)) if *p1 == *p2 => true,
333333

‎crates/hir_ty/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,7 @@ impl Ty {
300300
}
301301

302302
pub fn is_unknown(&self) -> bool {
303-
matches!(self.kind(&Interner), TyKind::Unknown)
303+
matches!(self.kind(&Interner), TyKind::Error)
304304
}
305305

306306
pub fn equals_ctor(&self, other: &Ty) -> bool {

‎crates/hir_ty/src/lower.rs

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ impl<'a> TyLoweringContext<'a> {
176176
let inner_ty = self.lower_ty(inner);
177177
TyKind::Ref(lower_to_chalk_mutability(*mutability), inner_ty).intern(&Interner)
178178
}
179-
TypeRef::Placeholder => TyKind::Unknown.intern(&Interner),
179+
TypeRef::Placeholder => TyKind::Error.intern(&Interner),
180180
TypeRef::Fn(params, is_varargs) => {
181181
let substs =
182182
Substitution::from_iter(&Interner, params.iter().map(|tr| self.lower_ty(tr)));
@@ -253,12 +253,12 @@ impl<'a> TyLoweringContext<'a> {
253253
data.provenance == TypeParamProvenance::ArgumentImplTrait
254254
})
255255
.nth(idx as usize)
256-
.map_or(TyKind::Unknown, |(id, _)| {
256+
.map_or(TyKind::Error, |(id, _)| {
257257
TyKind::Placeholder(to_placeholder_idx(self.db, id))
258258
});
259259
param.intern(&Interner)
260260
} else {
261-
TyKind::Unknown.intern(&Interner)
261+
TyKind::Error.intern(&Interner)
262262
}
263263
}
264264
ImplTraitLoweringMode::Variable => {
@@ -280,11 +280,11 @@ impl<'a> TyLoweringContext<'a> {
280280
}
281281
ImplTraitLoweringMode::Disallowed => {
282282
// FIXME: report error
283-
TyKind::Unknown.intern(&Interner)
283+
TyKind::Error.intern(&Interner)
284284
}
285285
}
286286
}
287-
TypeRef::Error => TyKind::Unknown.intern(&Interner),
287+
TypeRef::Error => TyKind::Error.intern(&Interner),
288288
};
289289
(ty, res)
290290
}
@@ -328,7 +328,7 @@ impl<'a> TyLoweringContext<'a> {
328328
(self.select_associated_type(res, segment), None)
329329
} else if remaining_segments.len() > 1 {
330330
// FIXME report error (ambiguous associated type)
331-
(TyKind::Unknown.intern(&Interner), None)
331+
(TyKind::Error.intern(&Interner), None)
332332
} else {
333333
(ty, res)
334334
}
@@ -372,12 +372,12 @@ impl<'a> TyLoweringContext<'a> {
372372
}
373373
None => {
374374
// FIXME: report error (associated type not found)
375-
TyKind::Unknown.intern(&Interner)
375+
TyKind::Error.intern(&Interner)
376376
}
377377
}
378378
} else if remaining_segments.len() > 1 {
379379
// FIXME report error (ambiguous associated type)
380-
TyKind::Unknown.intern(&Interner)
380+
TyKind::Error.intern(&Interner)
381381
} else {
382382
let dyn_ty = DynTy {
383383
bounds: Binders::new(
@@ -433,7 +433,7 @@ impl<'a> TyLoweringContext<'a> {
433433
self.lower_path_inner(resolved_segment, it.into(), infer_args)
434434
}
435435
// FIXME: report error
436-
TypeNs::EnumVariantId(_) => return (TyKind::Unknown.intern(&Interner), None),
436+
TypeNs::EnumVariantId(_) => return (TyKind::Error.intern(&Interner), None),
437437
};
438438
self.lower_ty_relative_path(ty, Some(resolution), remaining_segments)
439439
}
@@ -447,7 +447,7 @@ impl<'a> TyLoweringContext<'a> {
447447
let (resolution, remaining_index) =
448448
match self.resolver.resolve_path_in_type_ns(self.db.upcast(), path.mod_path()) {
449449
Some(it) => it,
450-
None => return (TyKind::Unknown.intern(&Interner), None),
450+
None => return (TyKind::Error.intern(&Interner), None),
451451
};
452452
let (resolved_segment, remaining_segments) = match remaining_index {
453453
None => (
@@ -498,9 +498,9 @@ impl<'a> TyLoweringContext<'a> {
498498
},
499499
);
500500

501-
ty.unwrap_or(TyKind::Unknown.intern(&Interner))
501+
ty.unwrap_or(TyKind::Error.intern(&Interner))
502502
} else {
503-
TyKind::Unknown.intern(&Interner)
503+
TyKind::Error.intern(&Interner)
504504
}
505505
}
506506

@@ -569,13 +569,13 @@ impl<'a> TyLoweringContext<'a> {
569569
def_generics.map_or((0, 0, 0, 0), |g| g.provenance_split());
570570
let total_len = parent_params + self_params + type_params + impl_trait_params;
571571

572-
substs.extend(iter::repeat(TyKind::Unknown.intern(&Interner)).take(parent_params));
572+
substs.extend(iter::repeat(TyKind::Error.intern(&Interner)).take(parent_params));
573573

574574
let fill_self_params = || {
575575
substs.extend(
576576
explicit_self_ty
577577
.into_iter()
578-
.chain(iter::repeat(TyKind::Unknown.intern(&Interner)))
578+
.chain(iter::repeat(TyKind::Error.intern(&Interner)))
579579
.take(self_params),
580580
)
581581
};
@@ -628,7 +628,7 @@ impl<'a> TyLoweringContext<'a> {
628628
// add placeholders for args that were not provided
629629
// FIXME: emit diagnostics in contexts where this is not allowed
630630
for _ in substs.len()..total_len {
631-
substs.push(TyKind::Unknown.intern(&Interner));
631+
substs.push(TyKind::Error.intern(&Interner));
632632
}
633633
assert_eq!(substs.len(), total_len);
634634

@@ -1008,7 +1008,7 @@ pub(crate) fn generic_defaults_query(
10081008
.enumerate()
10091009
.map(|(idx, (_, p))| {
10101010
let mut ty =
1011-
p.default.as_ref().map_or(TyKind::Unknown.intern(&Interner), |t| ctx.lower_ty(t));
1011+
p.default.as_ref().map_or(TyKind::Error.intern(&Interner), |t| ctx.lower_ty(t));
10121012

10131013
// Each default can only refer to previous parameters.
10141014
ty.walk_mut_binders(
@@ -1018,7 +1018,7 @@ pub(crate) fn generic_defaults_query(
10181018
// type variable default referring to parameter coming
10191019
// after it. This is forbidden (FIXME: report
10201020
// diagnostic)
1021-
*ty = TyKind::Unknown.intern(&Interner);
1021+
*ty = TyKind::Error.intern(&Interner);
10221022
}
10231023
}
10241024
_ => {}
@@ -1220,7 +1220,7 @@ pub(crate) fn ty_recover(db: &dyn HirDatabase, _cycle: &[String], def: &TyDefId)
12201220
TyDefId::AdtId(it) => generics(db.upcast(), it.into()).len(),
12211221
TyDefId::TypeAliasId(it) => generics(db.upcast(), it.into()).len(),
12221222
};
1223-
Binders::new(num_binders, TyKind::Unknown.intern(&Interner))
1223+
Binders::new(num_binders, TyKind::Error.intern(&Interner))
12241224
}
12251225

12261226
pub(crate) fn value_ty_query(db: &dyn HirDatabase, def: ValueTyDefId) -> Binders<Ty> {
@@ -1258,7 +1258,7 @@ pub(crate) fn impl_self_ty_recover(
12581258
impl_id: &ImplId,
12591259
) -> Binders<Ty> {
12601260
let generics = generics(db.upcast(), (*impl_id).into());
1261-
Binders::new(generics.len(), TyKind::Unknown.intern(&Interner))
1261+
Binders::new(generics.len(), TyKind::Error.intern(&Interner))
12621262
}
12631263

12641264
pub(crate) fn impl_trait_query(db: &dyn HirDatabase, impl_id: ImplId) -> Option<Binders<TraitRef>> {

‎crates/hir_ty/src/method_resolution.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -742,7 +742,7 @@ fn fallback_bound_vars(s: Substitution, num_vars_to_keep: usize) -> Substitution
742742
&mut |ty, binders| {
743743
if let TyKind::BoundVar(bound) = ty.kind(&Interner) {
744744
if bound.index >= num_vars_to_keep && bound.debruijn >= binders {
745-
TyKind::Unknown.intern(&Interner)
745+
TyKind::Error.intern(&Interner)
746746
} else {
747747
ty
748748
}

‎crates/hir_ty/src/op.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ pub(super) fn binary_op_return_ty(op: BinaryOp, lhs_ty: Ty, rhs_ty: Ty) -> Ty {
1515
| TyKind::Scalar(Scalar::Float(_)) => lhs_ty,
1616
TyKind::InferenceVar(_, TyVariableKind::Integer)
1717
| TyKind::InferenceVar(_, TyVariableKind::Float) => lhs_ty,
18-
_ => TyKind::Unknown.intern(&Interner),
18+
_ => TyKind::Error.intern(&Interner),
1919
}
2020
}
2121
BinaryOp::ArithOp(_) => match rhs_ty.kind(&Interner) {
@@ -24,7 +24,7 @@ pub(super) fn binary_op_return_ty(op: BinaryOp, lhs_ty: Ty, rhs_ty: Ty) -> Ty {
2424
| TyKind::Scalar(Scalar::Float(_)) => rhs_ty,
2525
TyKind::InferenceVar(_, TyVariableKind::Integer)
2626
| TyKind::InferenceVar(_, TyVariableKind::Float) => rhs_ty,
27-
_ => TyKind::Unknown.intern(&Interner),
27+
_ => TyKind::Error.intern(&Interner),
2828
},
2929
}
3030
}
@@ -37,10 +37,10 @@ pub(super) fn binary_op_rhs_expectation(op: BinaryOp, lhs_ty: Ty) -> Ty {
3737
TyKind::Scalar(_) | TyKind::Str => lhs_ty,
3838
TyKind::InferenceVar(_, TyVariableKind::Integer)
3939
| TyKind::InferenceVar(_, TyVariableKind::Float) => lhs_ty,
40-
_ => TyKind::Unknown.intern(&Interner),
40+
_ => TyKind::Error.intern(&Interner),
4141
},
4242
BinaryOp::ArithOp(ArithOp::Shl) | BinaryOp::ArithOp(ArithOp::Shr) => {
43-
TyKind::Unknown.intern(&Interner)
43+
TyKind::Error.intern(&Interner)
4444
}
4545
BinaryOp::CmpOp(CmpOp::Ord { .. })
4646
| BinaryOp::Assignment { op: Some(_) }
@@ -50,7 +50,7 @@ pub(super) fn binary_op_rhs_expectation(op: BinaryOp, lhs_ty: Ty) -> Ty {
5050
| TyKind::Scalar(Scalar::Float(_)) => lhs_ty,
5151
TyKind::InferenceVar(_, TyVariableKind::Integer)
5252
| TyKind::InferenceVar(_, TyVariableKind::Float) => lhs_ty,
53-
_ => TyKind::Unknown.intern(&Interner),
53+
_ => TyKind::Error.intern(&Interner),
5454
},
5555
}
5656
}

0 commit comments

Comments
 (0)