make linkrev non-optional

Summary:
I don't think it's possible for linkrevs to be `-1` (not sure why it was
written as `!0` -- jsgf, any idea why?)

Reviewed By: StanislavGlebik

Differential Revision: D5581780

fbshipit-source-id: 69d685d863ee51adc846e889eb766f3fecc47681
This commit is contained in:
Siddharth Agarwal 2017-08-08 09:55:53 -07:00 committed by Facebook Github Bot
parent 372ea24b88
commit 758669577e

View File

@ -78,7 +78,7 @@ pub struct Entry {
pub compressed_len: u32, // compressed content size
pub len: Option<u32>, // size of final file (after applying deltas)
pub baserev: Option<RevIdx>, // base/previous rev for deltas (None if literal)
pub linkrev: Option<RevIdx>, // changeset id
pub linkrev: RevIdx, // changeset id
pub p1: Option<RevIdx>, // parent p1
pub p2: Option<RevIdx>, // parent p2
pub nodeid: NodeHash, // nodeid
@ -137,7 +137,7 @@ named!(pub indexng<Entry>,
compressed_len: compressed_length,
len: Some(uncompressed_length),
baserev: if baserev == !0 { None } else { Some(baserev.into()) },
linkrev: if linkrev == !0 { None } else { Some(linkrev.into()) },
linkrev: linkrev.into(),
p1: if p1 == !0 { None } else { Some(p1.into()) },
p2: if p2 == !0 { None } else { Some(p2.into()) },
nodeid: NodeHash::from_bytes(&hash[..20]).expect("bad bytes for sha"),
@ -168,7 +168,7 @@ named!(pub index0<Entry>,
compressed_len: compressed_length,
len: None,
baserev: if baserev == !0 { None } else { Some(baserev.into()) },
linkrev: if linkrev == !0 { None } else { Some(linkrev.into()) },
linkrev: linkrev.into(),
p1: if p1 == !0 { None } else { Some(p1.into()) },
p2: if p2 == !0 { None } else { Some(p2.into()) },
nodeid: NodeHash::from_bytes(&hash[..20]).expect("bad bytes for sha"),