Homec4science

Add PhabricatorHash::digestForIndex()

Authored by epriestley <git@epriestley.com> on Dec 21 2012, 14:43.

Description

Add PhabricatorHash::digestForIndex()

Summary: Does this seem reasonable? It's a bit more compact than digest() (6 bits / byte instead of 4 bits / byte) and 72 bits of entropy @ 12 bytes instead of 128 bits of entropy @ 32 bytes. I feel like it's important to preserve the printability, though, and this seemed like a fairly good balance of concerns.

Test Plan: unit tests

Reviewers: vrana

Reviewed By: vrana

CC: aran, yemao932

Differential Revision: https://secure.phabricator.com/D4253

Details

Committed
epriestley <git@epriestley.com>Dec 21 2012, 14:43
Pushed
aubortJan 31 2017, 17:16
Parents
rPH141318fd01f4: Fix a Maniphest issue where multiple saved copies of a query break the menu
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH4af2e3c4e2e0: Add PhabricatorHash::digestForIndex() (authored by epriestley <git@epriestley.com>).Dec 21 2012, 14:43