On Sun, Mar 29, 2015 at 02:18:54PM +0200, Anton Khirnov wrote:
> --- a/libavcodec/hevc.c
> +++ b/libavcodec/hevc.c
> @@ -383,24 +383,63 @@ static int decode_lt_rps(HEVCContext *s, LongTermRPS
> *rps, GetBitContext *gb)
> return 0;
> }
>
> +static void export_stream_params(AVCodecContext *avctx,
> + const HEVCContext *s, const HEVCSPS *sps)
indentation
> +{
> + const HEVCVPS *vps = (const HEVCVPS*)s->vps_list[sps->vps_id]->data;
space before *
> + avctx->pix_fmt = sps->pix_fmt;
> + avctx->coded_width = sps->width;
> + avctx->coded_height = sps->height;
> + avctx->width = sps->output_width;
> + avctx->height = sps->output_height;
> + avctx->has_b_frames = sps->temporal_layer[sps->max_sub_layers -
> 1].num_reorder_pics;
> +
> - s->avctx->coded_width = sps->width;
> - s->avctx->coded_height = sps->height;
> - s->avctx->width = sps->output_width;
> - s->avctx->height = sps->output_height;
> - s->avctx->has_b_frames = sps->temporal_layer[sps->max_sub_layers
> - 1].num_reorder_pics;
Is there a reason to suddenly set pix_fmt as well? It sure looks odd in
a refactoring-only patch.
The amount of space before = is slightly odd, not that I care.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
