search

filecoin-project / builtin-actors

The Filecoin built-in actors
Other
78 stars 74 forks source link

Update Set and SetMultimap to use Map2 internally and match new conventions #1524

Closed anorth closed 4 months ago

anorth commented 4 months ago

Re #1346

anorth commented 4 months ago

Thanks for the suggestions, @rvagg. I rewrote some of the comments more fully, and deleted some that are now low value.

Stebalien commented 4 months ago

I played around with this a bit and I think we can get rid of the clone: https://github.com/filecoin-project/builtin-actors/pull/1525

(although I'm not sure if that'll work in all cases?)