Skip to content

Commit 65839fa

Browse files
committed
Emit a type error for integer literals where the expected type is char
For example, in let x: char = 42; This was an ICE and is now a proper type error, as per #3477
1 parent 5c6e928 commit 65839fa

File tree

3 files changed

+21
-1
lines changed

3 files changed

+21
-1
lines changed

src/librustc/middle/ty.rs

+14-1
Original file line numberDiff line numberDiff line change
@@ -144,10 +144,11 @@ export kind_is_durable;
144144
export meta_kind, kind_lteq, type_kind, type_kind_ext;
145145
export operators;
146146
export type_err, terr_vstore_kind;
147-
export terr_mismatch, terr_onceness_mismatch;
147+
export terr_integer_as_char, terr_mismatch, terr_onceness_mismatch;
148148
export type_err_to_str, note_and_explain_type_err;
149149
export expected_found;
150150
export type_needs_drop;
151+
export type_is_char;
151152
export type_is_empty;
152153
export type_is_integral;
153154
export type_is_numeric;
@@ -696,6 +697,7 @@ enum type_err {
696697
terr_in_field(@type_err, ast::ident),
697698
terr_sorts(expected_found<t>),
698699
terr_self_substs,
700+
terr_integer_as_char,
699701
terr_no_integral_type,
700702
terr_no_floating_point_type,
701703
}
@@ -2520,6 +2522,13 @@ fn type_is_integral(ty: t) -> bool {
25202522
}
25212523
}
25222524
2525+
fn type_is_char(ty: t) -> bool {
2526+
match get(ty).sty {
2527+
ty_int(ty_char) => true,
2528+
_ => false
2529+
}
2530+
}
2531+
25232532
fn type_is_fp(ty: t) -> bool {
25242533
match get(ty).sty {
25252534
ty_infer(FloatVar(_)) | ty_float(_) => true,
@@ -3489,6 +3498,10 @@ fn type_err_to_str(cx: ctxt, err: &type_err) -> ~str {
34893498
~"couldn't determine an appropriate integral type for integer \
34903499
literal"
34913500
}
3501+
terr_integer_as_char => {
3502+
~"integer literals can't be inferred to char type \
3503+
(try an explicit cast)"
3504+
}
34923505
terr_no_floating_point_type => {
34933506
~"couldn't determine an appropriate floating point type for \
34943507
floating point literal"

src/librustc/middle/typeck/infer/unify.rs

+4
Original file line numberDiff line numberDiff line change
@@ -362,6 +362,10 @@ impl infer_ctxt {
362362
}
363363

364364
fn int_var_sub_t(a_id: ty::IntVid, b: ty::t) -> ures {
365+
if ty::type_is_char(b) {
366+
return Err(ty::terr_integer_as_char);
367+
}
368+
365369
assert ty::type_is_integral(b);
366370

367371
let vb = &self.int_var_bindings;

src/test/compile-fail/issue-3477.rs

+3
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
fn main() {
2+
let _p: char = 100; //~ ERROR mismatched types: expected `char` but found
3+
}

0 commit comments

Comments
 (0)