Volume.cc: message maps

the diffstat speaks for itself.
You are overdoing it. This is much harder to understand than the nested ifs. I would go for a lighter refactoring:
- Swap the ifs so that `doing` is the inner one.
- use a single `sformat` like elsewhere

Then we don't have to think about
- using integers for bool
- the needless negation