This repository has been archived by the owner on Jan 4, 2024. It is now read-only.

remove send/sync retrictions when possible #12

Open

TyOverby wants to merge 2 commits into michaelwoerister:master from TyOverby:remove-reqs

TyOverby commented Sep 14, 2017

Because HamtMap isn't Send/Sync, there's no reason to make K or V Send or Sync.

This removes the necessity in the API, while allowing for the possibility of a Sync/Send HamptMap in the future.

TyOverby added 2 commits

September 14, 2017 15:06


          remove send/sync retrictions when possible

6a265e3


          fix rand dependency

c9209d4

Eh2406 commented Mar 7, 2018

I am still getting

     = help: the trait `std::marker::Sync` is not implemented for `std::rc::Rc<...>`
     = note: required by `hamt_rs::ShareStore`

with hamt-rs = { git = "https://github.com/TyOverby/hamt-rs", branch = "remove-reqs"}. Am I doing something wrong?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet