is this doing the right thing?