Hello,

Thanks for the comments.

> +
> > +    av_log(bsf, AV_LOG_DEBUG, "Texture to keep : %d.\n", ctx->texture);
>
> There's no point printing this on every packet. Either add an init()
> callback function and put it there, or remove it altogether since it's
> of dubious usefulness.
>

Removed.


>
>
> > +    ret = av_new_packet(out, target_packet_size);
> > +    if (ret < 0)
> > +        goto fail;
> > +
> > +    out_buf = out->data;
> > +    bytestream_put_buffer(&out_buf, in->data + start_section_size,
> target_packet_size);
> > +
> > +    ret = av_packet_copy_props(out, in);
> > +    if (ret < 0)
> > +        goto fail;
>
> Why not just do something like
>
> av_packet_move_ref(out, in);
>
> out->data += start_section_size;
> out->size  = target_packet_size;
>

Work for me, changed.

New patchs in attach, with these changes.

Martin

Attachment: 0001-avcodec-hap-move-parse_section_header-to-hap.c-in-or.patch
Description: Binary data

Attachment: 0002-avcodec-hapqa_extract_bsf-add-new-bsf-filter.patch
Description: Binary data

Attachment: 0003-doc-bsf-add-doc-for-new-hapqa_extract-bsf-filter.patch
Description: Binary data

Attachment: 0004-changelog-add-entry-for-hapqa-extract-bsf.patch
Description: Binary data

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to