On 2014-10-12 21:37, Colin Finck wrote:
> tfa...@svn.reactos.org wrote:
>> - DbgPrint(DbgString);
>> + OutputDebugStringA(DbgString);
>
> FYI, we had OutputDebugStringA there in the first place, but I changed
> it to DbgPrint in r40147.
> IIRC, output from DbgPrint was directly sent to the debug port while
> OutputDebugStringA buffers the data first. This is especially bad, when
> a test crashes the OS and we lose several kilobytes of log data before
> they are printed out.
>
> Probably, a DbgPrint("%s", DbgString) is the better alternative here.
Yeah that's the other alternative; however it doesn't seem needed?
Testbot is looking fine from what I see, and in a quick look at the
implementation I don't see any of the buffering you mention. Maybe
that's changed as well in the meantime?
I'm happy to change it if there are any problems with OutputDebugString.
_______________________________________________
Ros-dev mailing list
Ros-dev@reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev
