Defect with automating replay restarts

[GoogleCodeExporter]

What steps will reproduce the problem?
1. When trying to automate restarts on a series of replays using the suggested 
commands it misses out certain replays. E.g. given a set of replays 1.rep, 
2.rep, 3.rep, 4.rep it will always run 2.rep and 4.rep missing out 1 replay 
each time.

Main commands:
[auto_menu]
; auto_menu = OFF | SINGLE_PLAYER | LAN | BATTLE_NET
auto_menu = SINGLE_PLAYER

; auto_restart = ON | OFF
auto_restart = ON

; map = path to map relative to Starcraft folder
map = Maps\REPLAYS\*.rep

; mapiteration =  RANDOM | SEQUENCE
; type of iteration that will be done on a map name with a wildcard
mapiteration = SEQUENCE

I have attached the full file for reference.

What is the expected output? What do you see instead?
1. To have all replays execute (i.e. not to miss out one each time) in sequence.

What version of the product are you using? On what operating system?
BWAPI 3.7.4 on Windows 7 (x64), compiled using VC++ 2008 Express

Original issue reported on code.google.com by mdsumne...@gmail.com on 5 Nov 2013 at 6:59

Attachments:

• bwapi.ini

[GoogleCodeExporter]

Since BWAPI 3.4 is no longer being maintained (and no longer in the HEAD 
revision trunk), you should follow these steps:

1. Check out revision 4160 (3.7.4).
2. Apply the attached patch that fixes the issue.

I have yet to test this in the HEAD revision.

Original comment by AHeinerm on 5 Nov 2013 at 9:17

• Changed state: Started
• Added labels: Type-Defect, Priority-Critical, Component-Logic, Usability

Attachments:

• auto_map_sequence_fix.patch

[GoogleCodeExporter]

Note: You only need to compile the specific project "BWAPI" and its 
dependencies, you do NOT need to compile any of the other projects in the 
solution. You only need to replace the BWAPI.dll file generated. This will save 
you some time.

Issue is still open because verification in HEAD revision required.

Original comment by AHeinerm on 5 Nov 2013 at 9:19

[GoogleCodeExporter]

Fixed in BWAPI 4 revision 4457

Original comment by AHeinerm on 6 Nov 2013 at 3:08

• Changed state: Fixed