[Aaron]

Originally Posted by Minous

It looks right, but I haven't tested it. I also wouldn't use abValue for basic variable usage.

I would do something like:

var int_bonus as number
int_bonus = hero.child[aINT].field[aBonus].value
foreach pick in hero where wCategory.Firearm
       eachpick.field[dmrBonus].value += int_bonus 
nexteach

I will also note that the fields you are using may or may not be correct, I dont have HL handy to confirm.

In my opinion, its always much better to store a value used in a field than in a variable. Fields can be manipulated from the outside, while variables can't. For the same reason it's also best practices to split your eval scripts so that the first sets values, and the second uses those values, with some time in between in case something needs to intervene.