topic: remove validator safeguard that wasn't safeguarding anything The else block was simply setting origvalidator to None, but it's not used unless hg is version <= 5.3, but in that case origvalidator is set to tr._validator. So the block wasn't necessary, but also we can't call None as a function, so it didn't make a lot of sense either way. Also adding return to return the original validator return value. Mostly for consistence, because validators either raise an exception or they don't, return value currently doesn't seem to be used.