Added CC_FORMAT_PRINTF to CCString

Added for CCString::initWithFormat() and CCString::createWithFormat().
Fixed warnings.
This commit is contained in:
Sergey Shambir 2013-04-19 13:56:27 +04:00
parent e5b2f300f9
commit 3ae98978d4
4 changed files with 14 additions and 6 deletions

View File

@ -54,7 +54,7 @@ public:
CCString& operator= (const CCString& other); CCString& operator= (const CCString& other);
/** init a string with format, it's similar with the c function 'sprintf' */ /** init a string with format, it's similar with the c function 'sprintf' */
bool initWithFormat(const char* format, ...); bool initWithFormat(const char* format, ...) CC_FORMAT_PRINTF(2, 3);
/** convert to int value */ /** convert to int value */
int intValue() const; int intValue() const;
@ -95,7 +95,7 @@ public:
* @return A CCString pointer which is an autorelease object pointer, * @return A CCString pointer which is an autorelease object pointer,
* it means that you needn't do a release operation unless you retain it. * it means that you needn't do a release operation unless you retain it.
*/ */
static CCString* createWithFormat(const char* format, ...); static CCString* createWithFormat(const char* format, ...) CC_FORMAT_PRINTF(1, 2);
/** create a string with binary data /** create a string with binary data
* @return A CCString pointer which is an autorelease object pointer, * @return A CCString pointer which is an autorelease object pointer,

View File

@ -45,7 +45,6 @@ http://www.angelcode.com/products/bmfont/ (Free, Windows only)
using namespace std; using namespace std;
NS_CC_BEGIN NS_CC_BEGIN
// The return value needs to be deleted by CC_SAFE_DELETE_ARRAY. // The return value needs to be deleted by CC_SAFE_DELETE_ARRAY.
@ -153,8 +152,8 @@ CCBMFontConfiguration::~CCBMFontConfiguration()
const char* CCBMFontConfiguration::description(void) const char* CCBMFontConfiguration::description(void)
{ {
return CCString::createWithFormat( return CCString::createWithFormat(
"<CCBMFontConfiguration = %08X | Glphys:%d Kernings:%d | Image = %s>", "<CCBMFontConfiguration = " CC_FORMAT_PRINTF_SIZE_T " | Glphys:%d Kernings:%d | Image = %s>",
this, (size_t)this,
HASH_COUNT(m_pFontDefDictionary), HASH_COUNT(m_pFontDefDictionary),
HASH_COUNT(m_pKerningDictionary), HASH_COUNT(m_pKerningDictionary),
m_sAtlasName.c_str() m_sAtlasName.c_str()

View File

@ -259,6 +259,12 @@ public: virtual void set##funName(varType var) \
#define CC_FORMAT_PRINTF(formatPos, argPos) #define CC_FORMAT_PRINTF(formatPos, argPos)
#endif #endif
#if defined(_MSC_VER)
#define CC_FORMAT_PRINTF_SIZE_T "%08lX"
#else
#define CC_FORMAT_PRINTF_SIZE_T "%08zX"
#endif
#ifdef __GNUC__ #ifdef __GNUC__
#define CC_UNUSED __attribute__ ((unused)) #define CC_UNUSED __attribute__ ((unused))
#else #else

View File

@ -130,7 +130,10 @@ bool CCGLProgram::initWithVertexShaderFilename(const char* vShaderFilename, cons
const char* CCGLProgram::description() const char* CCGLProgram::description()
{ {
return CCString::createWithFormat("<CCGLProgram = %08X | Program = %i, VertexShader = %i, FragmentShader = %i>", this, m_uProgram, m_uVertShader, m_uFragShader)->getCString(); return CCString::createWithFormat("<CCGLProgram = "
CC_FORMAT_PRINTF_SIZE_T
" | Program = %i, VertexShader = %i, FragmentShader = %i>",
(size_t)this, m_uProgram, m_uVertShader, m_uFragShader)->getCString();
} }
bool CCGLProgram::compileShader(GLuint * shader, GLenum type, const GLchar* source) bool CCGLProgram::compileShader(GLuint * shader, GLenum type, const GLchar* source)