[FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states

Lou Logan lou at lrcd.com
Sat Oct 22 21:41:29 EEST 2016


On Sat, Oct 22, 2016, at 09:49 AM, Bodecs Bela wrote:
>
> under review: someone marked it because he/she investigate this patch. 
> So the patch submitter and other potential reviewers may feel/be_sure 
> that this patch is already handled by someone else.
> I suggest to use it. This is psychologic aspect that the patch submitter 
> may feel more patienty toward review process opposite to mere "new"
> state.

In my opinion that seems like unnecessary extra work. All patches in
theory are to be "under review". Reviewing takes enough time,
initiative, and motivation as is, and adding another, potentially
superfluous step just over complicates it.

Perhaps "Under Review" can just be removed, disabled, hidden, or
documented as "this option is ignored". Same with "Deferred" and
"Awaiting Upstream".


More information about the ffmpeg-devel mailing list