<div dir="ltr"><div dir="ltr">Hi Zeb,<div><br></div><div>Thanks for the review.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Op di 25 aug. 2020 om 17:16 schreef Zebediah Figura <<a href="mailto:z.figura12@gmail.com">z.figura12@gmail.com</a>>:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
A brief test shows that the function crashes if you pass NULL time while<br>
there's a valid clock, so I think you can just get rid of this check.</blockquote><div><br></div><div>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). </div><div><br></div><div>Kind regards,</div><div>Gijs </div></div></div>