[PATCH 1/2] dmime/tests: Added tests for IDirectMusicPerformance. (try 2)
Vitaliy Margolen
wine-devel at kievinfo.com
Thu Aug 12 00:36:27 CDT 2010
On 08/11/2010 11:16 PM, Austin Lund wrote:
> On 12 August 2010 14:32, Vitaliy Margolen<wine-devel at kievinfo.com> wrote:
>>
>> What are those magic numbers? Please use constants. If they are missing, add
>> them to appropriate header file first.
>
> They are the numbers passed to the InitAudio method from bug 22598.
1 is DMUS_APATH_SHARED_STEREOPLUSREVERB, 0x3f is DMUS_AUDIOF_ALL.
> I've seen quite a few places where "magic" numbers are used in tests
> with no references. I thought it would be ok to start the test off
> this way.
If they are a defined constant it's not a good thing to have them even in
tests. Of course you won't have a constant for for things like 128 in your
test (which is a number of performance channels).
> What is the best way to resubmit for another round of review? To
> wine-patches again (try 3)? Or on wine-devel? Or something else.
Send to wine-patches, as long as you have all the problems fixed. Just wait
a day to see if anyone else has any additional questions.
Vitaliy.
More information about the wine-devel
mailing list