I want to add the extra logging so the else is not useless. I'd prefer to leave the redundant assignment, just so the function is locally correct. I'd prefer the correctness of the function not depended on having people understand all the calling contexts. If there were a bigger performance hit, I might reconsider.