"Francois Gouget" fgouget@free.fr wrote:
- I scanned the source for TRACE/WARN/ERR/FIXME statements without a
terminating newline. There are few cases where this is justified because of following DPRINTFs. But most of the time there should be a \n otherwise the trace will not be flushed to disk immediately and the logs will look bad.
You forgot to remove redundant ' around %s in the traces when output is made by debugstr_a or debugstr_w. Actually you removed some of them, but not all.