On Thu, Jul 30, 2020 at 22:58:58 +0800, [email protected] wrote:
> if (codec_id == AV_CODEC_ID_AC3) {
> + int len = 1;
> +
> + if (ts_st->desc6a.component_type_flag) len++;
> + if (ts_st->desc6a.bsid_flag) len++;
> + if (ts_st->desc6a.mainid_flag) len++;
> + if (ts_st->desc6a.asvc_flag) len++;
This could be coded as
int len = 1
+ !!(ts_st->desc6a.component_type_flag)
+ !!(ts_st->desc6a.bsid_flag)
+ ...
but I don't know what is preferred (and technically, it results in the
same amount of machine commands). It probably doesn't matter.
> + side_data = (AVDescriptor6A *) av_packet_get_side_data(pkt,
> + AV_PKT_DATA_MPEGTS_DESC_6A, &sd_size);
Indentation is incorrect.
Cheers,
Moritz
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".