Copying one image to another using BitmapData

Question

5.00/5 (1 vote)

See more:

public Bitmap Background(Bitmap source)
        {
            width = source.Width;
            height = source.Height;

            BitmapData srcData = source.LockBits(new Rectangle(0, 0, width, height),
                ImageLockMode.ReadOnly, source.PixelFormat);

            byte* src = (byte*)srcData.Scan0.ToPointer();

            dstImage = new Bitmap(width, height,       PixelFormat.Format8bppIndexed);

            BitmapData dstData = dstImage.LockBits(new Rectangle(0, 0, width, height),
                ImageLockMode.ReadWrite, dstImage.PixelFormat);

            byte* dst = (byte*)dstData.Scan0.ToPointer();

            for (int y = 0; y < height; y++)
            {
                for (int x = 0; x < width; x++, src++, dst++)
                {
                    *dst = *src;
                }
                src = src + srcData.Stride - srcData.Width;     
                dst =  dst + dstData.Stride - dstData.Width;
            }
            source.UnlockBits(srcData);
            dstImage.UnlockBits(dstData);
            return dstImage;
        }

Hi,

I have this piece of code (please see above) to try to copy one image into another. The original image is grayscale but the final image obtained is colored. After several days of debugging, I realized that the R, G and B components of the final image are not equal while the initial image have equal RGB. My questions are in three parts:

1. What is wrong with the code below
2. The initial and the final images are Format8bppIndexed and I wasn't expecting it to have different channels for R, G and B (it even still has an alpha components)
3. Can someone please refer me to documents or articles describing the detailed structures of Format8bppIndexed image

Thank you.

Posted 13-Dec-10 2:48am

Akinola Falola

Updated 13-Dec-10 6:40am

Abdul Quader Mamun

v4

Add a Solution

Comments

Abdul Quader Mamun 13-Dec-10 12:40pm

Spelling check.

1 solution

Add a Solution

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

johannesnestler · Accepted Answer · 2010-12-13T04:00:00

You have to copy the color palette too! dstImage.Palette = source.Palette;Btw. you should fix your pointers when using unsafe code!
http://msdn.microsoft.com/en-us/library/f58wzh21(VS.80).aspx[^]
I used the following version of your snippet:

public static unsafe Bitmap CopyToNewBitmap(Bitmap source)
        {
            int width = source.Width;
            int height = source.Height;
            BitmapData srcData = source.LockBits(new Rectangle(0, 0, width, height), ImageLockMode.ReadOnly, source.PixelFormat);
            byte* src = (byte*)srcData.Scan0.ToPointer();
            Bitmap dstImage = new Bitmap(width, height, PixelFormat.Format8bppIndexed);

            dstImage.Palette = source.Palette; // copy color palette too!
            BitmapData dstData = dstImage.LockBits(new Rectangle(0, 0, width, height), ImageLockMode.ReadWrite, dstImage.PixelFormat);
             byte* dst = (byte*)dstData.Scan0.ToPointer();
            for (int y = 0; y < height; y++)
            {
                for (int x = 0; x < width; x++, src++, dst++)
                {
                    *dst = *src;
                }
                src = src + srcData.Stride - srcData.Width;
                dst = dst + dstData.Stride - dstData.Width;
            }
            source.UnlockBits(srcData); dstImage.UnlockBits(dstData);
            return dstImage;
        }

Copying one image to another using BitmapData

1 solution

Solution 2

Add your solution here

Preview 0

Copying one image to another using BitmapData

1 solution

Solution 2

Add your solution here

Preview 0

Existing Members

...or Join us