Page MenuHomePhabricator

The poop emoji (U+1F4A9) breaks differential
AbandonedPublic

Authored by NorthIsUp on Nov 1 2013, 9:10 PM.
Tags
None
Referenced Files
F18739240: D7472.id.diff
Wed, Oct 1, 8:30 PM
F18734515: D7472.id.diff
Tue, Sep 30, 10:55 PM
F18731835: D7472.diff
Tue, Sep 30, 3:19 PM
F18514368: D7472.id.diff
Sep 5 2025, 10:10 AM
F18480695: D7472.diff
Sep 3 2025, 11:21 AM
F18420314: D7472.diff
Aug 30 2025, 4:08 PM
F18106816: D7472.diff
Aug 10 2025, 11:24 PM
F17948831: D7472.id.diff
Jul 31 2025, 10:06 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T4045: Store diffs as binary, not UTF-8
Test Plan

not a real diff

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Although for the special case of diffs themselves (vs comment text, titles, summaries, etc.) I think we're going to move toward binary storage + runtime mangling.

Just pushing this out of my queue; see T4045 for strategy if you're feeling brave about fixing this.