Skip to content

Commit

Permalink
ir: We can't guarantee the type to be in the item map while propagati…
Browse files Browse the repository at this point in the history
…ng AlreadyResolved.

The item may come from a recursive check down the stack, and as we say there:

> Unchecked because we haven't finished this type yet.

Fixes rust-lang#1127
  • Loading branch information
emilio committed Oct 31, 2017
1 parent 3d623d3 commit 9c7f101
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/ir/item.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1512,7 +1512,7 @@ impl ClangItemParser for Item {
let result = Type::from_clang_ty(id, ty, location, parent_id, ctx);
let relevant_parent_id = parent_id.unwrap_or(current_module);
let ret = match result {
Ok(ParseResult::AlreadyResolved(ty)) => Ok(ty.expect_type_id(ctx)),
Ok(ParseResult::AlreadyResolved(ty)) => Ok(ty.as_type_id_unchecked()),
Ok(ParseResult::New(item, declaration)) => {
ctx.add_item(
Item::new(
Expand Down

0 comments on commit 9c7f101

Please sign in to comment.