On Thu, Jul 30, 2020 at 22:58:58 +0800, lance.lmw...@gmail.com 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 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".