On Wed, Dec 12, 2012 at 09:30:02PM +0100, Janne Grunau wrote:
> --- a/libavcodec/h264.c
> +++ b/libavcodec/h264.c
> @@ -2340,6 +2340,52 @@ int ff_h264_get_profile(SPS *sps)
> + if (s->avctx->has_b_frames > 1 || h->delayed_pic[0])
> + av_log(h->s.avctx, AV_LOG_WARNING, "Delayed frames seen
> reenabling "
> + "low delay requires a codec flush.\n");
This is slightly ungrammatical. I suspect what you want to say is
Delayed frames seen. Reenabling low delay requires a codec flush.
> + if (s->avctx->bits_per_raw_sample != h->sps.bit_depth_luma ||
> + h->cur_chroma_format_idc != h->sps.chroma_format_idc) {
> + if (s->avctx->codec &&
> + s->avctx->codec->capabilities & CODEC_CAP_HWACCEL_VDPAU
> + && (h->sps.bit_depth_luma != 8 || h->sps.chroma_format_idc > 1))
> {
> + av_log(s->avctx, AV_LOG_ERROR,
> + "VDPAU decoding does not support video colorspace\n");
nit: move && to the previous line as above and end the sentence in a period.
> @@ -2432,7 +2478,16 @@ static int decode_slice_header(H264Context *h,
> H264Context *h0)
> h->pps.sps_id);
> return -1;
> }
> - h->sps = *h0->sps_buffers[h->pps.sps_id];
> +
> + if (h->pps.sps_id != h->current_sps_id ||
> + h0->sps_buffers[h->pps.sps_id]->new) {
> + h0->sps_buffers[h->pps.sps_id]->new = 0;
> + h->current_sps_id = h->pps.sps_id;
> + h->sps = *h0->sps_buffers[h->pps.sps_id];
nit: align
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
