8000 Suggested change for TreeOp::Put · Issue #12 · turbofish-org/merk · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
Suggested change for TreeOp::Put #12
New issue
New issue
Closed
Closed
Suggested change for TreeOp::Put#12
Copy link
@davebryson

Description

@davebryson
davebryson
opened on Jul 14, 2019

Not a big deal, but one suggestion I have is to changeTreeOp::Put(&'a[u8]) to TreeOp::Put(Vec<u8>). Why?

  1. The consumer Node.value is already a Vec<u8>
  2. It simplifies TreeOp a bit (lifetime is not needed)
  3. It's easier to use Put(Vec<u8>) from external calls - (again simplifies lifetime stuff)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions
      0