Changing the Bid-BidPortfolio relationship that much would require a pretty substantive refactor of the existing code. Using bare pointers and const references would effectively just be a speed optimization. If it's required after some profiling exercises, I'd entertain such a refactor. I don't think this should hold up the PR.