On 16/04/15 11:55, Alexandra Hájková wrote:
> response to the last two Anton's reviews
>
> The old one is the result of the reverse engineering and guesswork.
> The new one has been written following the now-available specification.
>
> This work is part of Outreach Program for Women Summer 2014 activities
> for the Libav project.
>
> The fate references had to be changed because the old demuxer truncates
> the last frame in some cases, the new one handles it properly.
> The seek-test reference is changed because seeking works differently
> in the new demuxer. When seeking, the packet is not read from the stream
> directly, but it is rather constructed by the demuxer. That is why
> position is -1 now in the reference.
> ---
Line 680 and 693
av_log(s, AV_LOG_WARNING, "Suspicious data found! Stream will be
ignored.\n");
Mention which stream will be ignored.
Line 1089
p = asf_pkt->avpkt.data + asf_pkt->data_size - asf_pkt->size_left;
if ((p + pay_len) > (asf_pkt->avpkt.data + asf_pkt->data_size)) {
isn't the same as
pay_len > asf_pkt->size_left
Line 1459
av_clip_c -> av_clip ?
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
