cocos2dx 2.x版本在android下CCLabelTTF的一个bug

cocos2dx在android下是采用Paint来生成图片然后在CCLabelTTF里显示的，它具体的代码都在java类Cocos2dxBitmap里，生成完成之后会调用一个jni函数将结果传给cpp层，cpp层靠一个static变量来与java层交换数据，具体如下

       BitmapDC &dc = sharedBitmapDC();

        CC_BREAK_IF(! dc.getBitmapFromJava(pText, nWidth, nHeight, eAlignMask, pFontName, nSize));

        // assign the dc.m_pData to m_pData in order to save time
        m_pData = dc.m_pData;
        CC_BREAK_IF(! m_pData);

这里有个问题，sharedBitmapDC是一直共用的，它的dc.m_pData永远保存的是上一次的数据，如果某种原因java层调用的时候失败了，那么cpp层继续的话，就会拿到上上次的数据，但是这个数据是由cpp层负责free的，于是就会出现double free导致程序崩溃。我们在实际中就碰到了这个情况，当时崩溃的栈告诉我们的是CCImage析构里出现问题

CCImage::~CCImage()
{
    CC_SAFE_DELETE_ARRAY(m_pData);
#if (CC_TARGET_PLATFORM == CC_PLATFORM_WINRT) || (CC_TARGET_PLATFORM == CC_PLATFORM_WP8)
    CC_SAFE_DELETE(m_ft);
#endif
}

可是单纯看CCImage的代码是看不问题的，而且如果这里有问题，那麻烦大了。最终检查了之后还是确认cpp部分可能会隐藏bug，于是将上面CCImage里修改如下

        BitmapDC &dc = sharedBitmapDC();

        CC_BREAK_IF(! dc.getBitmapFromJava(pText, nWidth, nHeight, eAlignMask, pFontName, nSize));

        // assign the dc.m_pData to m_pData in order to save time
        m_pData = dc.m_pData;
        dc.m_pData = NULL;
        CC_BREAK_IF(! m_pData);

这样一来，cpp层肯定不会再出现double free了，于是再测，这个时候发现java层抛异常了，之前的异常由于程序退出根本就没打印得出来。再去追查java层，发现具体原因是

setContentSize之后，java层根据ContentSize判断出来，字体根本放不进去，于是它认为创建出来的Bitmap的高度是0，而Bitmap根本不允许创建高度为0的，于是异常了。我解决的办法是至少让它显示出来一行一列，这样我们看到显示就知道哪里出问题了，而不是直接崩溃。

从这个问题再一次印证了，fail early, fail loud的编程习惯，否则一个简单问题就变成复杂问题了。