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
F13812508: D9780.diff
Wed, Sep 18, 3:07 PM
Unknown Object (File)
Sun, Sep 8, 7:59 PM
Unknown Object (File)
Wed, Sep 4, 5:19 PM
Unknown Object (File)
Sun, Aug 25, 11:25 PM
Unknown Object (File)
Sun, Aug 25, 5:32 AM
Unknown Object (File)
Aug 16 2024, 8:15 PM
Unknown Object (File)
Aug 16 2024, 7:13 AM
Unknown Object (File)
Aug 13 2024, 8:45 PM
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).