rustc_metadata: Switch module children decoding to an iterator

Previously rust-lang/rust#103578, rust-lang/rust#103524 and previous PRs simplified it as much as possible.

A couple of cleanup commits is also added.
r? `@cjgillot`