🚀 go-pugleaf

In article <7328a61a.0310092311.7ba03fa1@posting.google.com>,
 heroin_c@hotmail.com (wall) wrote:

> I use this code selected the rect which has red eyes.
> 
> // the selected rect which has red eye
> // the selected image part GWorld - theSelectImageGWorld
> LGWorld*	theSelectImageGWorld = NEW LGWorld( theSelectRect,32 );
> ThrowIfNil_( theSelectImageGWorld ); 
> 
> // the orignal image GWorld - theImageGWorld
> PixMapHandle	theOPixMapH = ::GetGWorldPixMap(
> theImageGWorld->GetMacGWorld() );
> PixMapHandle	theSelectPixMapH = ::GetGWorldPixMap(
> theSelectImageGWorld->GetMacGWorld() );
> Ptr theOBaseAddr = ::GetPixBaseAddr( theOPixMapH );
> Ptr theSelectBaseAddr = ::GetPixBaseAddr( theSelectPixMapH );
> short  theORowBytes = ::GetPixRowBytes( theOPixMapH );
> short  theSelectRowBytes = ::GetPixRowBytes( theSelectPixMapH );
> 
> ::LockPixels( theOPixMapH );
> ::LockPixels( theSelectPixMapH );
> {
> 	Ptr		theSrcPtr, theDstPtr;
> 	for(short theY = mSelRectToImageOrignal.top; theY <
> mSelRectToImageOrignal.bottom; theY++)
> 	{
> 		theSrcPtr = theOBaseAddr + theORowBytes * theY;
> 		theSrcPtr += mSelRectToImageOrignal.left << 2;
> 		theDstPtr = theSelectBaseAddr + theSelectRowBytes * ( theY -
> mSelRectToImageOrignal.top );
> 		for (short theX = mSelRectToImageOrignal.left; theX <
> mSelRectToImageOrignal.right; theX++)
> 		{
> 			* ((long*) theDstPtr++) = * ((long*) theSrcPtr++);
> 			* ((long*) theDstPtr++) = * ((long*) theSrcPtr++);
> 			* ((long*) theDstPtr++) = * ((long*) theSrcPtr++);
> 			* ((long*) theDstPtr++) = * ((long*) theSrcPtr++);
> 
> 		}	
> 	}		
> }
> ::UnlockPixels( theOPixMapH );
> ::UnlockPixels( theSelectPixMapH );
> 
> what can I do to removing red eye in the selected image part?

As long as you can be relatively sure that the user has selected a close 
boundary around the pupil, you could just reduce the red level 
everywhere to the average of the green and blue values. This will help 
establish a reasonable intensity for the red channel and will capture 
any residual hilights that might be present. You may still need to bring 
down the overall intensity of each channel by some fixed percentage to 
avoid overly bright pupils.

Each 32-bit pixel is layed out as argb, 8 bits each. You should leave 
the a value alone.

Chris.