CloneCounter: Unnecessary wakes

[divergentdave]

I noticed that CloneCounter::decrement() unconditionally notifies all wakers in the WakerSet. If CloneCounterFuture.target were eliminated, (it is always zero) then the decrement() method could instead only notify when previously is equal to one, and save some unneeded wakes.

trillium/server-common/src/clone_counter.rs

Line 78 in 8dc1d8c

self.0.wake();

[jbr]

I haven't addressed this because I need to convince myself that this is thread safe and that with seqcst /some/ thread will always see the specific value in order to ensure waking happened