25 Aug
2020
25 Aug
'20
6:17 p.m.
Hi Zeb, Thanks for the review. Op di 25 aug. 2020 om 17:16 schreef Zebediah Figura <z.figura12(a)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