Skip to content
Snippets Groups Projects

Resolve "Refractor wotb to have separate traits for WoT memory layout and WoT calculations."

Closes #43 (closed)

Edited by nanocryk

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
150 } else {
151 buffer_3b.push(byte);
152 if count_bytes % 3 == 2 {
153 let mut buf = &buffer_3b.clone()[..];
154 let source = buf.read_u24::<BigEndian>().expect("fail to parse source");
155 wot.add_link(NodeId(source as usize), NodeId((target - 1) as usize));
156 remaining_links -= 1;
157 buffer_3b.clear();
158 }
159 count_bytes += 1;
160 }
161 }
162 Ok((wot, file_pointing_to_blockstamp))
163 }
164
165 /// Tru to write a `WebOfTrust` in a file.
  • @nanocryk Okay, that's perfect for me, with the exception of 2 points :

    • one typo
    • fmt fail at home, please upgrade your fmt and apply it

    then you can merge :)

    Edited by Éloïs
  • Author Contributor

    @librelois I applied the latest table fmt (via rustup) and only commited wotb formats, while warnings came from other crates outside of this MR scope.

  • @nanocryk ok for fmt but please correct the comment typo ;)

  • nanocryk added 1 commit

    added 1 commit

    Compare with previous version

  • Author Contributor
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading