Mac Forum / Programming / Mac Programming / October 2004

I am using NPickColor (as opposed to PickColor or GetColor) because IM
suggests that it is the most recent and most preferred).  The first time I
call it, works perfectly.  The second time, every time I run the program,
the program crashes.  This is what I've got:

pascal Boolean ColorPickerEventProc(EventRecord *event)
{
    return false;
}

pascal void ColorPickerColorChanged(SInt32 userData, NPMColor *newColor)
{
}

void SomeFunction()
{
    NColorPickerInfo cpi;
    cpi.theColor.color.rgb.red = currentColor.red;
    cpi.theColor.color.rgb.green = currentColor.green;
    cpi.theColor.color.rgb.blue = currentColor.blue;
    cpi.dstProfile = NULL;    //Use default profile
    cpi.placeWhere = kAtSpecifiedOrigin;    //Doesn't seem to work,
but oh well!
    cpi.dialogOrigin = where;    //Doesn't seem to work, but oh
well!
    cpi.pickerType = 0;    //Use last used type
    UserEventUPP eventProcUPP = NewUserEventUPP(ColorPickerEventProc);
    cpi.eventProc = eventProcUPP;
    NColorChangedUPP colorChangedUPP =
NewNColorChangedUPP(ColorPickerColorChanged);
    cpi.colorProc = colorChangedUPP;
    cpi.colorProcData = 0;    //user data passed to color changed
callback
    cpi.prompt[0] = 0;    //This dialog has no text prompt
    cpi.mInfo.editMenuID = 130;
    cpi.mInfo.cutItem = 4;
    cpi.mInfo.copyItem = 5;
    cpi.mInfo.pasteItem = 6;
    cpi.mInfo.clearItem = 7;
    cpi.mInfo.undoItem = 1;

    if (NPickColor(&cpi) == noErr && cpi.newColorChosen)
    {
        //process new color
    }

    DisposeUserEventUPP(eventProcUPP);
    DisposeNColorChangedUPP(colorChangedUPP);
}

It's hard to describe the crash.  It's an access fault exception that
occurs deep inside the colorsync routines, a long ways from my code.

If anyone knows how to get this working properly, I would appreciate it.

Thank you very much.

________________________________________________________________________
Keith Wiley         kwiley@cs.unm.edu         http://www.unm.edu/~keithw

"Yet mark his perfect self-contentment, and hence learn his lesson,
that to be self-contented is to be vile and ignorant, and that to
aspire is better than to be blindly and impotently happy."
                                           --  Edwin A. Abbott, Flatland
________________________________________________________________________