Re: [PATCH 3/3] dmsynth: Put port caps that match native ones.
Just to be clear, which "native" do you mean? Does this match Windows 7's dmusic behavior, or something else? I've been trying to target Windows 7's behavior in the new audio design. Also... On Thu, Mar 29, 2012 at 08:53:52AM +0200, Christian Costa wrote:
- obj->pCaps.dwEffectFlags = DMUS_EFFECT_REVERB | DMUS_EFFECT_CHORUS | DMUS_EFFECT_DELAY; + obj->pCaps.dwEffectFlags = DMUS_EFFECT_REVERB; /* DMUS_EFFECT_CHORUS | DMUS_EFFECT_DELAY */
I'd just remove the unused flags. No need for crufty comments.
2012/3/29 Andrew Eikum <aeikum(a)codeweavers.com>
Just to be clear, which "native" do you mean? Does this match Windows 7's dmusic behavior, or something else? I've been trying to target Windows 7's behavior in the new audio design.
They're recents but I don't remember. I will check this evening. I didn't know about Windows 7 target but I'm ok with that so I will aligned if needed.
Also...
On Thu, Mar 29, 2012 at 08:53:52AM +0200, Christian Costa wrote:
- obj->pCaps.dwEffectFlags = DMUS_EFFECT_REVERB | DMUS_EFFECT_CHORUS | DMUS_EFFECT_DELAY; + obj->pCaps.dwEffectFlags = DMUS_EFFECT_REVERB; /* DMUS_EFFECT_CHORUS | DMUS_EFFECT_DELAY */
I'd just remove the unused flags. No need for crufty comments.
Ok I will remove them.
participants (2)
-
Andrew Eikum -
Christian Costa