Stefano Sabatini:
> +static void print_frame_side_data(WriterContext *w,
> + AVFrame *frame,
> + AVStream *stream)
I am pretty sure both frame and stream can be constified.
> +{
> + int i;
We support C99 variable declarations in for loops.
> +
> + writer_print_section_header(w, SECTION_ID_FRAME_SIDE_DATA_LIST);
> +
> + for (i = 0; i < frame->nb_side_data; i++) {
> + AVFrameSideData *sd = frame->side_data[i];
This can probably be const, too.
> + const char *name;
_______________________________________________
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".