Reduce the number of things that are "pub"
kevinlewi opened this issue · comments
Right now, there are way too many functions / structs that are pub, which I think do not need to be. We should reduce this as much as possible, preferring pub(crate).
On a similar note: anything that is only meant to be called by tests should ideally have a _for_testing() prefix, and at least be marked as [cfg(test)]