Rather than having
```
#ifndef __MINGW32__ // negative test
...
#else // double-negative: not not MinGW, i.e. MinGW.
...
#endif
```
it would be nicer to use the positive test and put the (brief) MinGW-specific block first.  (This would also make it more obvious to the reader why we're treating MinGW specially, as the Log() message explains all.)  It is generally better to write positive tests, when you're going to have an else clause and cover both halves anyway, since readers are human and humans have some cognitive deficiencies when it comes to double negation.