`solvate_topology` and `solvate_topology_nonwater` are a little bit confusing

[IAlibay]

Note: this is very low priority, in fact changing things would be more work for me 🙃

This isn't a bug per se, just an observation whilst adding a path for neutralizing systems with pontibus.

solvate_topology is effectively "the path you take if you want to neutralize", i.e. the code is such that neutralization will happen no matter what. So technically if you don't want to use pack_box and you don't want to neutralize, you should be using solvate_topology_nonwater whether or not you are using a water solvent.

This issue is just pointing to the latter maybe being a tad bit confusing to users.

[mattwthompson]

So technically if you don't want to use pack_box and you don't want to neutralize

IIUC this would produce a non-neutral system