On Thu, Nov 21, 2019 at 08:08:54 -0300, ggarr...@gmail.com wrote: > From: Gonzalo Garramuño <ggarr...@gmail.com> > Subject: [FFmpeg-devel] [PATCH] Added interrupt to av_read_frame() in > utils.c, based on a patch by bsenftner at earthlink.net. This allows > av_read_frame to return after a timeout period.
Small nit regarding the commit message: Please prefix the first line with the affected "module", i.e. "avformat/utils", and keep the first line short. Then add an empty line(!), then the remaining text. You also don't need to describe the actual act of adding something. E.g. as such: --- avformat/utils: allow av_read_frame to return after a timeout period Based on a patch by bsenftner at earthlink.net. --- (Also, if this fixes some trac ticket, add a line/paragraph stating "Fixes #12345.") 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".
