Shoudln't this be a void method?