Hi,

> Hi Viktor,
> 
> This solution strips the trailing isspace characters from the debug info and
> inserts the necessary CR/LF for windbg, not more.
> Maybe we can define here a lot of similar solutions for the same thing.

One problem with this is that it's now implemented as a 
Windows-only solution. (BTW I don't really understand why 
Windows debugging needs are _always_ different from any 
other platforms...). Overall I'm not sure this function 
should alter passed trace string in any ways.

Proper fix IMO is to change those trace calls which pass 
ending spaces. There is probably not too many of them.

As for the anti-reentrance logic, it probably needs to be 
changed to mutexes, as in ST I can't see any way it can be 
called in recursive fashion, and in MT current solution will 
just delay problems by causing lost trace messages.

Brgds,
Viktor

_______________________________________________
Harbour mailing list (attachment size limit: 40KB)
Harbour@harbour-project.org
http://lists.harbour-project.org/mailman/listinfo/harbour

Reply via email to