|| -#if defined(_MSC_VER) && _MSC_VER < 1300
|| +#ifdef _MSC_VER
|| #define __func__ "??"
|
| I thought some versions of VC *did* have an acceptable version of
| __func__ or a replacement for it.

Yes, __FUNCTION__, as libevent already uses.

| What is _MSC_VER defined to on VC6?

1200.

_MSC_VER 1200: MSC 12.0, VC 6.0, VS 1998
_MSC_VER 1300: MSC 13.0, VC 7.0, VS 2002
_MSC_VER 1310: MSC 13.1, VC 7.1, VS 2003
_MSC_VER 1400: MSC 14.0, VC 8.0, VS 2005
_MSC_VER 1500: MSC 15.0, VC 9.0, VS 2008

http://predef.sourceforge.net/precomp.html

| Should the test be something other than _MSC_VER < 1300?

I'm not sure when MSC first added __FUNCTION__ support. It is present in VS2005. The below 'WinCE 5.0 Platform Builder' MSDN excerpt seems to agree with your current (pre-patch) test:

----snip----
The new compiler includes the _FUNCTION_ predefined macro. For backward compatible code, add the following definition:

#if _MSC_VER < 1300
#define __FUNCTION__ ""
#endif
----snip----




_______________________________________________
Libevent-users mailing list
Libevent-users@monkey.org
http://monkeymail.org/mailman/listinfo/libevent-users

Reply via email to