[PATCH v3 2/3] amstream: Implement IAMMultiMediaStream::GetDuration.
z.figura12 at gmail.com
Tue Jul 28 10:43:28 CDT 2020
On 7/28/20 3:53 AM, Myah Caron wrote:
>> It seems, given the tests below, that S_FALSE is returned if the graph
>> can't seek (i.e. it's translated from E_NOTIMPL). In that case it'd be
>> better to key this return value on the return from AddMediaStream() above.
> Thanks for the review.
> Sorry if I'm misunderstanding you, but I'm pretty sure the earlier AddMediaStream calls succeed under windows server. I believe the issue is that decoding AVI files isn't supported under windows server (I read something to that effect a few days ago, though for some reason I'm unable to find the link now).
We render and seek an AVI file (which admittedly is video-only) in
quartz:filtergraph, apparently successfully:
I'd expect OpenFile() to fail if the file can't be rendered.
On the other hand, w2008s uniquely lacks audio hardware, so I would
indeed expect VFW_E_NO_AUDIO_HARDWARE when adding the default audio
> Would adding an extra test (like IAMMultiMediaStream_GetInformation) to see if AVI files can be seeked be alright?
>> Also, a bit of a nitpick, but I think that duration would be more
>> readable in decimal.
> Good point, will fix.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 488 bytes
Desc: OpenPGP digital signature
More information about the wine-devel