C / C++ / MFC

toxcct wrote:

man, it should be this :

if (Channel == -1)

by using = instead of ==, you're assigning it !

VonHagNDaz wrote:

this is a feature within the vs2005 debugger.

if (-1 == Channel)

VonHagNDaz wrote:

for( Channel = 0; Channel < NumberOfChannels; Channel++ )

VonHagNDaz wrote:

memset(Out,0x00,OutLen);

VonHagNDaz wrote:

number of channels represents the differnt ink cartridges inside of the E*s*n printers we are modifying. 1 - 8 for dual CMYK printing...Out is the pixel data for that channel, and OutLen is the length of the data that is expected. say 16 pixel wide image, four ink channels, you would end up with four arrays each containing the color data for that specific channel for each pixel.

VonHagNDaz wrote:

Channel is an int and is set to zero at the start of the loop,

VonHagNDaz wrote:

...out is an unsigned char*...

VonHagNDaz wrote:

Outlen is and unsigned int.

VonHagNDaz wrote:

NumberOfChannels is an int.

VonHagNDaz wrote:

...and OutLen 2692.

VonHagNDaz wrote:

its just randomly that Channel = -1.

DavidCrow wrote:

They do exactly what they were instructed to do

VonHagNDaz wrote:

A coworker has suggested that it might be a memory error where something is writing to that chunk and skewing the number?

for (int Channel = 0; Channel < NumberOfChannels; Channel++)
{
    if (-1 == Channel)
        ; // set breakpoint on this line
    
    Dat = (unsigned char*)m_Dither.Dat[Channel];
    Out = (unsigned char*)lpDithered->Data[Channel]; 
    ChannelData = &(lpDthrCtrl->Data[Channel]);
    ...
}