same comments here, but to be honest, if you're going to reuse the xxx_SHIFT define somewhere else, I don't see any benefit in adding these