Page MenuHomePhabricator

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

Authored by NorthIsUp on Nov 1 2013, 9:10 PM.
Tags
None
Referenced Files
F19931795: D7472.id16843.diff
Fri, Apr 10, 4:13 AM
F19898732: D7472.diff
Sun, Mar 29, 3:29 PM
F19893934: D7472.diff
Sat, Mar 28, 10:45 AM
F19892936: D7472.diff
Sat, Mar 28, 5:16 AM
F19815224: D7472.id.diff
Mar 4 2026, 7:13 PM
F19814564: D7472.id.diff
Mar 4 2026, 2:33 PM
F19811295: D7472.id.diff
Mar 3 2026, 11:45 PM
F19153239: D7472.id16843.diff
Dec 11 2025, 8:32 AM
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.