does this need to be public? should it be?