On Tue, 27 Dec 2011, Andrey Utkin wrote:
понедельник, 26 декабря 2011 г. пользователь Martin Storsjö <[email protected]> писал: > > This might be ok, but this needs a check for v->n_segments > 0. To simplify the expression, a separate duration variable perhaps can be declared above, e.g. > > int duration = v->n-segments > 0 ? > v->segments[v->n_segments - 1]->duration : > v->target_duration; > > // MartinYes, such situation seems weird, but we don't have such check during parse_playlist. And wowza is sufficiently weird software to end up in such situation :) Looking more at that code, i expanded a patch. Please review again.
The change looks kinda good in itself, but the spec quote in the comment perhaps is a bit excessive. I cleaned it up a little, and implemented the other part of that spec section - switching to half the target duration if the playlist didn't update. Please test the two new patches I'll send in a second.
// Martin
_______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
