I've created a fork with a fix. I just wait that the PR #133 is merged as I use it in my version

I think the common solutions that other projects use are bad:

#ifdef _MSC_VER
#undef GetObject
#endif

This is bad as this can break user code that relies on GetObject WinAPI function.

#pragma push_macro("GetObject")
#undef GetObject

...

#pragma pop_macro("GetObject")

This is also bad as the user has no way of actually using GetObject as he would get either GetObjectA or GetObjectW if he doesn't explicitly #undef it in his code.

The best way would be temporarily undefining the macro with push_macro/pop_macro in addition to providing inline GetObjectA() and GetObjectW() functions that would just call a real GetObject() - this would work and nobody would notice a thing.

I tried here:

#139

I actually don't have a Windows machine here so it's a blind fix - hope it works!

@andy3469 Can you confirm the PR #139 works for you?

Is there anything that would block merging the PR?

I think only the method applied can be discussed - it has pros and cons, but mostly it works in all cases, no need introducing alternative function or forcing users to undef the macro.

Any news for the PR ?

@andy3469 I am not able to reproduce the error. Please refer PR #141 and respective discussion is in #139 (comment)

It's easy to reproduce the error.

Compile minio-cpp with UNICODE and use it from an app that doesn't define UNICODE or vice versa. Basically now you have to ensure to compile minio-cpp and all code that depends on it having the same UNICODE definition (either defined or not), which is a little bit problematic if you are offering a library for others to consume.

The problem is that if you include <windows.h> the GetObject macro WILL ALWAYS be defined, so in minio-cpp you would actually never have GetObject symbol - it would be either GetObjectA or GetObjectW depending on what <windows.h> defines.

@kobalicek I was actually typing the same thing.

When you add #define UNICODE and #include <windows.h> at the top of tests.cc with the main branch I can reproduce the error.