Modify reform time based on food usage have conflict

[jzy-chitong56]

When peon_assignment in UD, it will call FormGroupAM.
Does this conflict occur???

so I give some change

  if race_has_ghouls then
    call HarvestWood(0, Max(harvesting_ghouls,0))
    if GetWood() < 400 then
      call HarvestWood(0, Min(TownCountDone(racial_ghoul), 5))
    elseif take_all_ghouls_along then // no need harvest wood
      call AddAssault(attacking_ghouls,old_id[racial_ghoul])
    elseif peons_assigned and ghouls_in_attack then
      call HarvestWood(0, Max(TownCountDone(racial_ghoul) - attacking_ghouls - harvesting_ghouls,0)) // spare ghouls not doing anything and not assigned to attack yet
      call AddAssault(attacking_ghouls,old_id[racial_ghoul])
      //call FormGroupAM(0) // in an attack so don't let harvest AI override attack
    else
      call HarvestWood(0, Max(TownCountDone(racial_ghoul) - harvesting_ghouls,0)) // spare ghouls not doing anything
    endif
    set peons_assigned = true

[SMUnlimited]

True, it could also trigger the same pause to wait for units. I'll have to think about this case if there another way to do it, or perhaps I can test if it isn't needed anymore since I fixed the calculations on the attack side it might keep things in sync.

[SMUnlimited]

Above code is fine with take_all_ghouls_along as it pulls every ghoul off, but doesn't work when its not.

[SMUnlimited]

Either no or hardly any ghouls go, or all the ghouls go. Can't have the right number without formgroup running every time.

[SMUnlimited]

Got it to not form when no ghouls attacking, but nothing reliable in other approaches.