Proper fix for iterating over unit tags

Issue was returning by value was creating a copy of a set instead
of returning a reference (which was what I expected).
Attack boosts now properly save tags

I think I goofed this commit and part of the change is in the prior
commit, which probably won't build because of this.