btQuickprof.h including windows.h, and a possible solution

[LtJax]

Hello,

I just figured out that including bullet with some of my code breaks compilation. This seems to be due to bullet's btClock in btQuickprof.h which includes <windows.h> along with all its horrible defines.
Our current fix for this issue is to simply disable profiling (which seems to be the solution to some other portability problems with btQuickprof too). However, it should also be relatively easy to hide the inclusion of <windows.h> in the corresponding .cpp file, using the PIMPL idiom. If that is desired, I'll whip up a patch that does just that - but before I would like to know if that is a desireable solution.

Any arguments against such a solution? Is it even needed for somebody else?

Cheers,
Marius