Skip to content

Conversation

@barosl
Copy link
Contributor

@barosl barosl commented Dec 13, 2014

check::autoderef() returns a ty_err when it fails to infer the type. probe::probe() should respect this failure and fail together to prevent further corruption.

Fixes #19692.
Fixes #19583.
Fixes #19297.

@barosl barosl force-pushed the autoderef-type-inference-ice branch from aa01242 to bf3bbd2 Compare December 13, 2014 16:09
@barosl
Copy link
Contributor Author

barosl commented Dec 13, 2014

It seems that #19583 was also caused by this bug.

@barosl barosl force-pushed the autoderef-type-inference-ice branch 2 times, most recently from 34a8fee to d5cf71c Compare December 13, 2014 19:20
@alexcrichton
Copy link
Member

r? @nikomatsakis

@alexcrichton
Copy link
Member

or r? @pnkfelix (or other wizards in this area)

@barosl barosl force-pushed the autoderef-type-inference-ice branch from 18bc8a1 to df31e32 Compare December 19, 2014 19:53
check::autoderef() returns a ty_err when it fails to infer the type.
probe::probe() should respect this failure and fail together to prevent
further corruption.

Call stack: check::check_method_call() -> method::lookup() ->
            probe::probe() + confirm::confirm()

Fixes rust-lang#19692.
Fixes rust-lang#19583.
Fixes rust-lang#19297.
@barosl barosl force-pushed the autoderef-type-inference-ice branch from df31e32 to d9f3067 Compare December 19, 2014 19:54
bors added a commit that referenced this pull request Dec 24, 2014
…kfelix

`check::autoderef()` returns a `ty_err` when it fails to infer the type. `probe::probe()` should respect this failure and fail together to prevent further corruption.

Fixes #19692.
Fixes #19583.
Fixes #19297.
bors added a commit that referenced this pull request Dec 24, 2014
…kfelix

`check::autoderef()` returns a `ty_err` when it fails to infer the type. `probe::probe()` should respect this failure and fail together to prevent further corruption.

Fixes #19692.
Fixes #19583.
Fixes #19297.
@bors bors merged commit d9f3067 into rust-lang:master Dec 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

6 participants