cordyceps: Should `list::RawIter` impl Send?

[jamesmunns]

Reported in tweedegolf/cfg-noodle#14 (comment), holding the iterator across an await point (e.g. using an async mutex for the List) means that the RawIter might need to impl send. IMO this is probably fine since we are bound by the lifetime of the mut borrow of the list.

[jamesmunns]

(I think it probably should, but @hawkw tell me if I'm speaking crazy here)

[jamesmunns]

oh, and maybe another bug, I did make the method take a mutable reference for iter_raw, but the STRUCT doesn't contain an &mut ref:

https://github.com/hawkw/mycelium/blob/main/cordyceps/src/list.rs#L262

I wonder if Iter/IterMut would have the same issues being held across an await point. Might be worth checking if I push a fix for this.

[hawkw]

(I think it probably should, but @hawkw tell me if I'm speaking crazy here)

Yeah, it probably should

oh, and maybe another bug, I did make the method take a mutable reference for iter_raw, but the STRUCT doesn't contain an &mut ref:

Whoops, that should definitely be fixed; the list needs to be mutably borrowed as long as the iterator exists...