One adapter at a time per repository.

Not all commits in this patch build or pass tests separately.

rebase on main

refactor: use a type to capture non-zero count of concurrent adapters

Signed-off-by: Lars Wirzenius liw@liw.fi

refactor: use more idiomatic Rust to check result of operation

Signed-off-by: Lars Wirzenius liw@liw.fi

fix(src/queueproc.rs): rewrite to avoid unimplemented!

This should never trigger, but just in case, we don’t ever want it to trigger in production.

Signed-off-by: Lars Wirzenius liw@liw.fi

refactor: drop now-unnecessary pragma to allow unused imports

Signed-off-by: Lars Wirzenius liw@liw.fi

refactor: drop unnecessary pragma for allowing dead code

Signed-off-by: Lars Wirzenius liw@liw.fi

feat(src/queueproc.rs): log reason why waiting for adapter fails

Signed-off-by: Lars Wirzenius liw@liw.fi

refactor: drop commented-out constants

Signed-off-by: Lars Wirzenius liw@liw.fi

refactor(src/pull_queue.rs): avoid asserts

I didn’t like having asserts, and didn’t want debug_asserts. With plain asserts, the program would have panicked. With debug_asserts, the program would have silently overwritten the item in the queue. Instead, change the method to return whether it was able to set, and have the caller cope with not being able to set. The caller needs to be able handle that (and was already doing so).

Found-by: Fintan Signed-off-by: Lars Wirzenius liw@liw.fi

refactor(src/pull_queue.rs): drop “end” method, add Drop impl

This makes it harder to forget to close the producer end.

Suggested-by: Fintan Signed-off-by: Lars Wirzenius liw@liw.fi