[FFmpeg-devel] [PATCH] avformat/smoothstreamingenc: check return value of avcodec_parameters_copy()

Marth64 marth64 at proxyid.net
Thu Nov 21 09:02:12 EET 2024


Hi Zhao Zhili:

> We don’t use braces in such case generally.
> And please split assignment and check
> into two lines next time.

I agree but was trying to fix it in the style of the
surrounding code for consistency. The function
mostly uses braces after single quote and
the assignment/check pattern.

It doesn’t make it ideal but I also did not
want to make it look awkward.

Thank you


More information about the ffmpeg-devel mailing list