Inline short-living variables of `collectArgFromFormulas`

[kharus]

Set produced by collectArgFromFormulas is mostly iterated immediately in for loops apart from few places. This change converts one iteration of collectArgFromFormulas into foreach. And removes temp vars and moves usage of collectArgFromFormulas directily into foreach like in other places. Also forms argument generalized from ArrayList to List.

[apease]

I like removing iterators. I'm not sure about incorporating the calls to produce a list into the foreach. It seems that might have an impact on making debugging more difficult