Zhao, Gang (12020-05-02):
> With the new api, function write_audio_frame and write_video_frame can be
> combined to one.
> Because there is already a local function called write_frame, I changed the
> name to muxing_write_frame
> to avoid name collision.

Multiple changes in the same patch make it harder to review. You could
name the merged function write_any_frame().

And if you think it is best, make another patch to rename both
functions: it would be easier to review.

Regards,

-- 
  Nicolas George

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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".

Reply via email to