Page MenuHomePhabricator

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

Authored by NorthIsUp on Nov 1 2013, 9:10 PM.
Tags
None
Referenced Files
F19815224: D7472.id.diff
Wed, Mar 4, 7:13 PM
F19814564: D7472.id.diff
Wed, Mar 4, 2:33 PM
F19811295: D7472.id.diff
Tue, Mar 3, 11:45 PM
F19153239: D7472.id16843.diff
Dec 11 2025, 8:32 AM
F19092879: D7472.id.diff
Dec 3 2025, 6:57 PM
F19084332: D7472.diff
Dec 2 2025, 3:09 PM
F18857915: D7472.id.diff
Nov 1 2025, 7:04 PM
F18853303: D7472.diff
Oct 31 2025, 3:42 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.