Page MenuHomePhabricator

Fix an issue with symbol lookup identifying path names in Diffusion
ClosedPublic

Authored by epriestley on Jan 26 2018, 2:45 PM.
Tags
None
Referenced Files
F19321519: D18940.id45434.diff
Thu, Dec 25, 10:04 AM
F19081188: D18940.id.diff
Dec 2 2025, 4:23 AM
F18980469: D18940.diff
Nov 17 2025, 12:43 AM
F18814701: D18940.diff
Oct 20 2025, 11:47 PM
F18757228: D18940.id.diff
Oct 5 2025, 4:16 PM
F18754550: D18940.diff
Oct 4 2025, 11:28 PM
F18588433: D18940.diff
Sep 12 2025, 12:17 AM
F18566509: D18940.id.diff
Sep 9 2025, 2:41 PM
Subscribers
None

Details

Summary

Depends on D18939. Ref T13047. Symbol lookup can be activated from a diff (in Differential or Diffusion) or from the static view of a file at a particular commit.

In the latter case, we need to figure out the path a little differently. The character and line number approaches still work as written.

Test Plan
  • Command-clicked symbols in the Diffusion browse view with blame on and off; saw path, line and char populate properly.
  • Command-clicked symbols in Differential diff view to check I didn't break anything.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable