]> git.lizzy.rs Git - rust.git/commit
Merge #5518
authorbors[bot] <26634292+bors[bot]@users.noreply.github.com>
Fri, 24 Jul 2020 12:46:55 +0000 (12:46 +0000)
committerGitHub <noreply@github.com>
Fri, 24 Jul 2020 12:46:55 +0000 (12:46 +0000)
commitc3defe2532ba6ffd12a13bcbc8fdeda037665efc
tree831bf4dd44ec83d927face4ba17e57dcdeab7fbe
parent0e5095d3cac11d4b569c6e1594bd07937556c812
parent58680cb08ea535e1fb567416fa3466a744a01b99
Merge #5518

5518: Use resolved paths in SSR rules r=matklad a=davidlattimore

The main user-visible changes are:
* SSR now matches paths based on whether they resolve to the same thing instead of whether they're written the same.
  * So `foo()` won't match `foo()` if it's a different function `foo()`, but will match `bar::foo()` if it's the same `foo`.
* Paths in the replacement will now be rendered with appropriate qualification for their context.
  * For example `foo::Bar` will render as just `Bar` inside the module `foo`, but might render as `baz::foo::Bar` from elsewhere.
* This means that all paths in the search pattern and replacement template must be able to be resolved.
* It now also matters where you invoke SSR from, since paths are resolved relative to wherever that is.
* Search now uses find-uses on paths to locate places to try matching. This means that when a path is present in the pattern, search will generally be pretty fast.
* Function calls can now match method calls again, but this time only if they resolve to the same function.

Co-authored-by: David Lattimore <dml@google.com>