Re: AUCarbonViewControl::HandleEvent bug?
Re: AUCarbonViewControl::HandleEvent bug?
- Subject: Re: AUCarbonViewControl::HandleEvent bug?
- From: Airy André <email@hidden>
- Date: Thu, 6 Feb 2003 08:11:03 +0100
Le jeudi, 6 fiv 2003, ` 01:37 Europe/Paris, Marc Poirier a icrit :
In AUCarbonViewControl.cpp, in the AUCarbonViewControl::HandleEvent()
method, there is this:
case kEventControlClick:
if (mLastControl != this) {
if (mLastControl != NULL)
mLastControl->Update(false);
mLastControl = this;
} mOwnerView->TellListener(mParam,
kAudioUnitCarbonViewEvent_MouseDownInControl, NULL);
break; // don't return true, continue normal processing
case kEventControlHit:
if (mLastControl != this) {
if (mLastControl != NULL)
mLastControl->Update(false);
mLastControl = this;
mOwnerView->TellListener(mParam,
kAudioUnitCarbonViewEvent_MouseUpInControl, NULL);
}
break; // don't return true, continue normal processing
But shouldn't that actually be:
case kEventControlClick:
if (mLastControl != this) {
if (mLastControl != NULL)
mLastControl->Update(false);
mLastControl = this;
} mOwnerView->TellListener(mParam,
kAudioUnitCarbonViewEvent_MouseDownInControl, NULL);
break; // don't return true, continue normal processing
case kEventControlHit:
if (mLastControl != this) {
if (mLastControl != NULL)
mLastControl->Update(false);
mLastControl = this;
} mOwnerView->TellListener(mParam,
kAudioUnitCarbonViewEvent_MouseUpInControl, NULL);
break; // don't return true, continue normal processing
(notice the that in second to last line above, TellListener() is moved
outside of the if (mLastControl != this) statement)
I think that this is a bug and should be changed to the second version
above, but maybe I am misunderstanding the code. Either way, I would
appreciate a confirmation of which is correct, I'm deep in automation
woes
lately...
Marc
_______________________________________________
coreaudio-api mailing list | email@hidden
Help/Unsubscribe/Archives:
http://www.lists.apple.com/mailman/listinfo/coreaudio-api
Do not post admin requests to the list. They will be ignored.
I agree with that. I'd signaled the same problem in this list when the
SDK has been released
and proposed something similar.
There is also another problem that make automation not working with
discrete parameters :
In "Bind", the following correction should be made :
< if (mType == kTypeContinuous || mType == kTypeText) {
---
> if (mType == kTypeContinuous || mType == kTypeText || mType ==
kTypeDiscrete) {
Airy
_______________________________________________
coreaudio-api mailing list | email@hidden
Help/Unsubscribe/Archives:
http://www.lists.apple.com/mailman/listinfo/coreaudio-api
Do not post admin requests to the list. They will be ignored.