Fix vs2013 compilation - Githubissues

eranpeer / FakeIt

C++ mocking made easy. A simple yet very expressive, headers only library for c++ mocking.

MIT License

1.24k stars 170 forks source link

Fix vs2013 compilation #66

Closed amerry closed 8 years ago

amerry commented 8 years ago

This works around a compiler bug / lack of compiler support in Visual Studio 2013.

All tests pass on Visual Studio 2013, Visual Studio 2015, gcc 4.8.4 (Ubuntu 14.04) and AppleClang 7.3.0/703.0.31 (Mac OS X El Capitan).

Fixes #65.

coveralls commented 8 years ago

Coverage remained the same at 99.974% when pulling 0bfb8745426b0cacf12ae85c4680b68949463072 on amerry:fix_vs2013_compilation into dc844899e14a949e7abc3d6abb6cd423f8f79982 on eranpeer:master.

amerry commented 8 years ago

Note: I tried to make the behaviour as much the same in VS2013 as in other compilers, hence the overrides for ios setters. This doesn't necessarily make sense, though - you don't actually want to stream function arguments that are ios setters.

eranpeer commented 8 years ago

Thank's a lot. Nice & clean fix.