[ROSAUTOTEST]
authorThomas Faber <thomas.faber@reactos.org>
Tue, 23 Jun 2015 10:06:38 +0000 (10:06 +0000)
committerThomas Faber <thomas.faber@reactos.org>
Tue, 23 Jun 2015 10:06:38 +0000 (10:06 +0000)
commit634ccedc1cf57f8a5b22640fa05778f65a066d51
tree1524510ac87d42b4c1bf8638b6df37fa98f776bd
parent77ad885551a087c9ea1cf2eacb4132db0c9b16b5
[ROSAUTOTEST]
- Flush cout after each write as already done by Colin in r66855, per ROSTESTS-158
- Use DbgPrint instead of OutputDebugStringA again because the latter only calls the former anyway
Fixes test summary lines not being recognized by Testman, especially on VMware.

A little summary so we don't keep going back and forth with this function:
1) Only writing complete lines is required to that the output doesn't mix with debug output from other components. See r55618
2) OutputDebugStringA splits its input into 512-byte-sized blocks with no regard for line breaks, so using it with strings larger than 512 bytes breaks (1).
3) OutputDebugStringA eventually calls DbgPrint("%s", string) anyway so using it with chunks smaller than 512 bytes is not an optimization
As a result, yes this function MUST split up the lines itself, this can't be optimized or simplified away! kthxbye
ROSTESTS-178 #resolve

svn path=/trunk/; revision=68246
rostests/rosautotest/tools.cpp