This is an archive of the discontinued LLVM Phabricator instance.

Instead of scream, why not roll?
Needs RevisionPublic

Authored by lazerl0rd on Apr 1 2020, 3:30 PM.

Details

Reviewers
nathanchance
rsmith
Group Reviewers
Restricted Project
Summary

Everyone naturally converts hex to text when they see a long
value, so provide clearer information on their bad coding habits
with a nice link.

If you're still a bit lazy, why not use https://www.browserling.com/tools/hex-to-text?

Diff Detail

Event Timeline

lazerl0rd created this revision.Apr 1 2020, 3:30 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 1 2020, 3:30 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
lazerl0rd rescinded a token.
rsmith requested changes to this revision.Apr 2 2020, 5:10 PM
rsmith added a subscriber: rsmith.

I'm sorry, but I can't accept this patch as-is because it removes the u suffix from the literal. We're never gonna give u up.

This revision now requires changes to proceed.Apr 2 2020, 5:10 PM