clear() takes ownership, so it can't be wrapped

[borovan]

Hi there, this is our Store which uses a Wrapped stable BTreeMap

I wanted to wrap the clear() method but I'm unable to, so the workaround is just to go over every key and delete it. We've got tests with 10,000 inserts so this doesn't seem like a great way to go about it.

I've tried std::mem::take, but that doesn't work as there's no Default value for the map. There's not much I can do because the MemoryID is required and I can't keep a reference to it for some reason.

Any ideas on how to implement clear?

[ielashi]

Hi @borovan, thanks for bringing this up. I think it should be possible to implement clear without taking ownership. Let me take a look.

[dragoljub-djuric]

@borovan Thank you for the request, here is the PR that added the new method clear_new() which is in the new 0.6.3 release of Stable Structures, while clear is marked as deprecated.

[borovan]

Awesome, thanks for this!