Hi Zeb,
Thanks for the review.
Op di 25 aug. 2020 om 17:16 schreef Zebediah Figura z.figura12@gmail.com:
A brief test shows that the function crashes if you pass NULL time while there's a valid clock, so I think you can just get rid of this check.
While I definitely don't feel strongly about this, elsewhere in amstream we check for NULL on things that crash on native (IAMMediaStream::Initialize(), IMediaStreamFilter::GetCurrentStreamTime()/GetState to name a few).
Kind regards, Gijs