From: Andrew Eikum <aeikum@codeweavers.com>
Subject: Re: msacm32/tests: Add some acmStreamPrepareHeader and acmStreamConvert tests (try 2)
Message-Id: <20151124185113.GO9736@foghorn.codeweavers.com>
Date: Tue, 24 Nov 2015 12:51:13 -0600
In-Reply-To: <20151124184850.GN9736@foghorn.codeweavers.com>
References: <CALF_nUQf-gdas7OQPg=WAam3TQBy0+gULi46CTjbmo3+aHF4=Q@mail.gmail.com> <20151124184850.GN9736@foghorn.codeweavers.com>

On Tue, Nov 24, 2015 at 12:48:50PM -0600, Andrew Eikum wrote:
> On Tue, Nov 24, 2015 at 05:08:00PM +0800, Bruno Jesus wrote:
> > +    hdr.cbSrcLengthUsed = 12345;
> > +    hdr.cbDstLengthUsed = 12345;
> 
> The rest looks OK, but I don't understand why these are set. Does the
> application really set them to this value after Prepare and before
> Convert? The tests seem to behave the same if I leave them at 0.
> 

Oh, I see, the rest of the tests do that, too. Probably to demonstrate
that Convert changes the value. OK.

Signed-off-by: Andrew Eikum <aeikum@codeweavers.com>