Shouldn't you return in this case?