setFromIK should return true if the function succeeded if the IK call succeeded and we set the state to a valid state, according to the constraints we use (which could be empty). There is a bit of weirdness here in the sense that the state could be destroyed even if we return false. I agree with your points above, but if we can keep the ptr stuff a little more hidden for the stuff that can be const&, I'd be happy.