lance.lmw...@gmail.com:
> From: Limin Wang <lance.lmw...@gmail.com>
>
> Signed-off-by: Limin Wang <lance.lmw...@gmail.com>
> ---
> libavcodec/utils.c | 38 ++++++--------------------------------
> 1 file changed, 6 insertions(+), 32 deletions(-)
>
> diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> index e77090d..3ae23fc 100644
> --- a/libavcodec/utils.c
> +++ b/libavcodec/utils.c
> @@ -583,38 +583,12 @@ int attribute_align_arg avcodec_open2(AVCodecContext
> *avctx, const AVCodec *code
> }
> avctx->internal = avci;
>
> - avci->to_free = av_frame_alloc();
> - if (!avci->to_free) {
> - ret = AVERROR(ENOMEM);
> - goto free_and_end;
> - }
> -
> - avci->compat_decode_frame = av_frame_alloc();
> - if (!avci->compat_decode_frame) {
> - ret = AVERROR(ENOMEM);
> - goto free_and_end;
> - }
> -
> - avci->buffer_frame = av_frame_alloc();
> - if (!avci->buffer_frame) {
> - ret = AVERROR(ENOMEM);
> - goto free_and_end;
> - }
> -
> - avci->buffer_pkt = av_packet_alloc();
> - if (!avci->buffer_pkt) {
> - ret = AVERROR(ENOMEM);
> - goto free_and_end;
> - }
> -
> - avci->ds.in_pkt = av_packet_alloc();
> - if (!avci->ds.in_pkt) {
> - ret = AVERROR(ENOMEM);
> - goto free_and_end;
> - }
> -
> - avci->last_pkt_props = av_packet_alloc();
> - if (!avci->last_pkt_props) {
> + if (!(avci->to_free = av_frame_alloc()) ||
> + !(avci->compat_decode_frame = av_frame_alloc()) ||
> + !(avci->buffer_frame = av_frame_alloc()) ||
> + !(avci->buffer_pkt = av_packet_alloc()) ||
> + !(avci->ds.in_pkt = av_packet_alloc()) ||
> + !(avci->last_pkt_props = av_packet_alloc())) {
> ret = AVERROR(ENOMEM);
> goto free_and_end;
> }
>
I think this would look better if it were aligned on '='.
- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
