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

Authored by NorthIsUp on Nov 1 2013, 9:10 PM.

Tags

None

Referenced Files

	F15510037: D7472.id.diff
	Wed, Apr 16, 6:16 PM

	F15506990: D7472.diff
	Tue, Apr 15, 3:39 PM

	F15505261: D7472.diff
	Tue, Apr 15, 1:21 AM

	F15489230: D7472.diff
	Fri, Apr 11, 7:09 AM

	F15416545: D7472.diff
	Thu, Mar 20, 11:55 AM

	F15381050: D7472.id.diff
	Mar 14 2025, 5:38 AM

	Unknown Object (File)
	Feb 22 2025, 3:28 PM

	Unknown Object (File)
	Feb 10 2025, 5:56 AM

View All Files

Subscribers

aran

epriestley

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

See T1191.

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.

NorthIsUp abandoned this revision.Mar 4 2014, 9:02 PM

Revision Contents
Changeset List

Path

Size

jenkins/

phabricator-post.py

8 lines

Diff 16843

View Options

jenkins/phabricator-post.py

The poop emoji (U+1F4A9) breaks differentialAbandonedPublicActions