Naming clash in CCTouchDispatcher.cpp about cocos2d-x HOT 6 CLOSED

Did you mean that, less function should be put into namespace cocos2d?
cocos2d-x don't support Mac OS X, so you port it to Mac OS X?

from cocos2d-x.

Do you mind posting issues in official forum later?
The link is http://www.cocos2d-x.org/projects/cocos2d-x/boards.

Thank you.

from cocos2d-x.

Did you mean that, less function should be put into namespace cocos2d?

No fear, it's one-time function (as far as I understood), it should not pollute API namespace, it should be only accessible in that compilation unit.

I placed it into anonymous namespace, because it's (according to Stroustrup) preferred way to restrict function visibility:
“The use of the static keyword is deprecated when declaring objects in a namespace scope, the unnamed-namespace provides a superior alternative.”

cocos2d-x don't support Mac OS X, so you port it to Mac OS X?

No, I just building under Mac OS X targeting iOS 4.3. I've got a compilation error on XCode, but everything is OK on MSVS 2010 under Windows. There are actually some differences in #include'ing headers of “stable” libraries (Cocos2d-x, Loki, Boost, STL) in precompiled header in Visual C++ and prefix header in XCode.

from cocos2d-x.

Probable steps to reproduce (have not checked yet):

1. Create XCode project with Cocos2d-x template. Source code of Cocos2d-x will be included in xcodeproj directly. Therefore prefix header will be implicitly added to every compilation unit.
2. Edit Prefix.pch:

#ifdef __OBJC__
    #import <Foundation/Foundation.h>
    #import <UIKit/UIKit.h>
#elif defined __cplusplus
    #include <functional>
#endif

3. Try to build.

from cocos2d-x.

@Qbit86 : That PCH file is incorrect.  Both __OBJC__ and __cplusplus are both defined when a .cpp or .mm file is compiled. 
In other words, the conditions should be nested, instead of being disjunct.

from cocos2d-x.

@qbit86 Please, could you open a bug in redmine in future? We are in process of migrating all the issues to redmine and github issue won't be available after August

from cocos2d-x.