fix: add continue

[cuiweixie]

should be same in:

rust/compiler/rustc_trait_selection/src/error_reporting/infer/nice_region_error/static_impl_trait.rs

Lines 156 to 159 in d3ec6a3

	if fn_return.span.desugaring_kind().is_some() {
	// Skip `async` desugaring `impl Future`.
	continue;
	}

[rustbot]

r? @JonathanBrouwer

rustbot has assigned @JonathanBrouwer.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

• Owners of files modified in this PR: borrowck, compiler
• borrowck, compiler expanded to 22 candidates
• Random selection from 13 candidates

[JonathanBrouwer]

Hi there, thanks for your contribution!

Could you explain what bug this is fixing, and if possible, add a regression test?

[cuiweixie]

Hi there, thanks for your contribution!

Could you explain what bug this is fixing, and if possible, add a regression test?

I think add this continue statement is reasonable, same as in

rust/compiler/rustc_trait_selection/src/error_reporting/infer/nice_region_error/static_impl_trait.rs

Lines 156 to 159 in d3ec6a3

	if fn_return.span.desugaring_kind().is_some() {
	// Skip `async` desugaring `impl Future`.
	continue;
	}

[JonathanBrouwer]

This looks right to me, especially given the comment, but don't know the code and could also believe a break being better.
@rustbot reroll

[jieyouxu]

Question: can a test demonstrate the diagnostics effect before/after this change? The change itself looks reasonable.

[cuiweixie]

I have try to,construct. But do not find the right way. it failed then.

[petrochenkov]

@bors r+ rollup
Looks reasonable and only affects some corner cases in diagnostics, I'll just approve without a test.

[rust-bors]

📌 Commit d3ec6a3 has been approved by petrochenkov

It is now in the queue for this repository.