In my opinion, you should use the "for each integer" loop instead of a "while" loop. Then you won't have to manually increase or reset the variables. Like this:
(copied from another post of mine, but it's sort of the same thing)
Spawn units Events Game - Map initialization Local Variables x = 0.0 <Real> y = 0.0 <Real> Conditions Actions General - For each integer y from 0 to (Integer((Height of Spawn 01))) with increment 1, do (Actions) Actions General - For each integer x from 0 to (Integer((Width of Spawn 01))) with increment 1, do (Actions) Actions Unit - Create 1 Marine for player 1 at ((Bottom-left of Spawn 01 bounds) offset by (x, y)) using default facing (No Options)
what they said
Need a delete post button
<- Its right there to the left. (You can do it yourself)