Random thought: Any time you find yourself using `unreachable!(...)`, consider that there might be a better model for your abstraction.
I had this happen to me recently, and realized that I could break a struct out into an enum with two distinct cases, eliminating a branch
