sapling/eden/scm/lib/dag
Jun Wu 064068169b dag: rename slice to VertexName
Summary:
Per discussion, we decided to use "VertexName" as the struct name for things
like commit hashes, or the string names in tests (or the Mozilla DAG in tests).
Therefore, introduce a dedicated VertexName type and repalce all callsites to
use it.

`bytes::Bytes` is used so copying the `VertexName` is somewhat considered cheap.

This adds some overhead copying slices (and `Bytes` has some overhead). It
regresses the "building segments" benchmark from 673ms to 773ms, which seems
okay given the cleaner interface.

Reviewed By: markbt

Differential Revision: D19154905

fbshipit-source-id: 4c6d4eca67c11c10ed5f21999ccdc3f1b01695e8
2020-01-08 21:35:28 -08:00
..
benches dag: rename slice to VertexName 2020-01-08 21:35:28 -08:00
src dag: rename slice to VertexName 2020-01-08 21:35:28 -08:00
Cargo.toml dag: rename slice to VertexName 2020-01-08 21:35:28 -08:00