Page MenuHomePhabricator

Only show text encoding note in Differential if a change has hunks
ClosedPublic

Authored by epriestley on Jun 29 2014, 5:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 6, 7:03 PM
Unknown Object (File)
Tue, May 6, 2:35 PM
Unknown Object (File)
Sat, May 3, 3:38 PM
Unknown Object (File)
Mar 27 2025, 1:20 PM
Unknown Object (File)
Mar 22 2025, 7:42 AM
Unknown Object (File)
Mar 22 2025, 2:47 AM
Unknown Object (File)
Mar 19 2025, 6:22 PM
Unknown Object (File)
Mar 19 2025, 6:47 AM
Subscribers

Details

Summary

Fixes T5503. We incorrectly render an encoding note for empty files. Only render an encoding note for text changes with at least one hunk.

Test Plan
  • Viewed empty file, no note.
  • Viewed nonempty file with altered encoding, saw note.

Diff Detail

Repository
rP Phabricator
Branch
diff1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1462
Build 1462: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Only show text encoding note in Differential if a change has hunks.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: joshuaspence, btrahan.
joshuaspence edited edge metadata.

LGTM. Thanks

This revision is now accepted and ready to land.Jun 29 2014, 6:15 PM
epriestley updated this revision to Diff 23479.

Closed by commit rPc1af499ed7ab (authored by @epriestley).