oops wrong loop iterator