x/auction/keeper: IterateAuctions and other helpers should be returning pointers to unmarshalled objects to avoid awkward APIs that consume pointers yet receive values

[odeke-em]

Please update the iterator consumer to take auction pointers and not values because the consumers of the APIs end up needing pointers and that all becomes awkward