Page MenuHomePhabricator

Don't syntax highlight codebase pattern search results
ClosedPublic

Authored by epriestley on Mar 24 2015, 12:59 PM.
Tags
None
Referenced Files
F13094510: D12141.diff
Thu, Apr 25, 7:21 AM
Unknown Object (File)
Fri, Apr 19, 3:31 AM
Unknown Object (File)
Thu, Apr 11, 7:16 AM
Unknown Object (File)
Thu, Mar 28, 1:23 AM
Unknown Object (File)
Mar 5 2024, 8:00 AM
Unknown Object (File)
Feb 9 2024, 12:54 AM
Unknown Object (File)
Feb 5 2024, 9:18 AM
Unknown Object (File)
Feb 3 2024, 2:40 AM
Subscribers

Details

Summary

Ref T5644. Ref T7472. Currently, we highlight each line of pattern search results in Diffusion.

  • This is incredibly slow for non-PHP languages which need to shell out to Pygments.
  • A lot of this highlighting isn't very useful anyway, because it doesn't have any context.

Instead, try to highlight pattern matches but don't highlight the source itself.

Test Plan

Screen_Shot_2015-03-24_at_5.56.22_AM.png (1×1 px, 192 KB)

Diff Detail

Repository
rP Phabricator
Branch
highlight
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4956
Build 4974: [Placeholder Plan] Wait for 30 Seconds