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)
Fri, Dec 20, 11:52 AM
Unknown Object (File)
Fri, Dec 20, 5:55 AM
Unknown Object (File)
Tue, Dec 17, 5:28 PM
Unknown Object (File)
Mon, Dec 9, 2:28 PM
Unknown Object (File)
Mon, Dec 9, 12:42 PM
Unknown Object (File)
Wed, Dec 4, 9:10 AM
Unknown Object (File)
Tue, Dec 3, 2:10 PM
Unknown Object (File)
Sat, Nov 30, 1:39 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).