Bitmap copy bug?

[stanleyycho]

Hello,

I have been developing reader using libCZI. When I used this function. I was wondering if it the CBitmapOperations::Copy function should be copying from PixelType::Bgr48 to PixelType::Bgr48 using Copy<PixelType::Bgr48, PixelType::Bgr48> instead of Copy<PixelType::Bgr24, PixelType::Bgr24>.

The current implementation only copies half of the buffer.

in libCZI/BitmapOperations.cpp at line 131

/*static*/void CBitmapOperations::Copy(libCZI::PixelType srcPixelType, const void* srcPtr, int srcStride, libCZI::PixelType dstPixelType, void* dstPtr, int dstStride, int width, int height, bool drawTileBorder)
{
/*
 ...
*/
        case PixelType::Bgr48:
        switch (dstPixelType)
        {
        case PixelType::Gray8:break;
        case PixelType::Gray16:break;
        case PixelType::Gray32Float:break;
        case PixelType::Bgr24:break;
        case PixelType::Bgr48:
                        // should be Bgr48
            Copy<PixelType::Bgr48, PixelType::Bgr48>(srcPtr, srcStride, dstPtr, dstStride, width, height, drawTileBorder); 
            return;
        default:break;
        }
        break;

    default:break;
    }
/*
 ...
*/
}

[zeissmicroscopy]

Yes, sure. Thanks for reporting!