* #0: various fixes for windows build
* remove warnings that definition of a variable hides previously defined ones
* remove warnings that function argument is not used
* remove warnings that statements are not reached
* event_trace_win: fix compilation with UNICODE
* #0 various windows fixes: react to PR comments
* localize unicode handling of process info to ProcessInfo
* fix compilation error in Properties constructor
* event_trace_win: fix bug in ProcessInfo initialization
We didn't retry setting the name when GetModuleBaseName() didn't succeed,
even though we did that before the last refactoring.
* Revert "event_trace_win: fix bug in ProcessInfo initialization"
This reverts commit c4ce6cf488be5489fa0a1a9c24e92dd39bbc42a1.
* Revert "#0 various windows fixes: react to PR comments"
This reverts commit cae31a25371e53a3645ddab82372c3772138d658.
* #0 various windows fixes: react to PR comments again
* keep number of allocations minimal
* add getProcessName() to retrieve the process name as char always (i.e. not UNICODE)
* This required a minor change to CMakeLists.txt.
* Also updated two occurrences in code where compilation failed due to
implicit narrowing conversions.
Thread id changed to size_t, required for MacOS because older versions do not have integral thread ids and we must use a pointer returned by pthread_self()/