Skip to content

Commit

Permalink
Auto merge of #100865 - compiler-errors:parent-substs-still, r=cjgillot
Browse files Browse the repository at this point in the history
Don't drop parent substs when we have no generic parameters in `create_substs_for_ast_path`

This bug is being shadowed by an explicit check for `generics.params.is_empty()` in the only parent caller that could trigger it (`create_substs_for_associated_item`). I triggered it on another branch where I'm messing around with astconv stuff.

Also, the second commit simplifies `create_substs_for_associated_item`. Removing that explicit check I mentioned above^ and also the special case call to `Astconv::prohibit_generics` causes the UI test `src/test/ui/structs/struct-path-associated-type.stderr` to change, but I think that it's clearer now. The suggestion to remove the generics is actually useful.
  • Loading branch information
bors committed Sep 25, 2022
2 parents e93032f + efe7c16 commit 97bac81
Showing 0 changed files with 0 additions and 0 deletions.

0 comments on commit 97bac81

Please sign in to comment.