search

approvals / ApprovalTests.cpp

Native ApprovalTests for C++ on Linux, Mac and Windows
https://approvaltestscpp.readthedocs.io/en/latest/
Apache License 2.0
316 stars 51 forks source link

R Do not include Windows.h in SystemUtils.h #96

Closed p-podsiadly closed 4 years ago

p-podsiadly commented 4 years ago

Description

SystemUtils.h includes Windows.h which leads some annoyances like having to #define NOMINMAX etc.

The solution

This change replaces calls functions declared in <Windows.h> with functions from <io.h>, so that the header does not have to be included.

claremacrae commented 4 years ago

Thanks @p-podsiadly !