packagestructures: Use use for loop with string ref instead of mutable iterator

This is simpler and we don't need to construct a new string