[FFmpeg-devel] [PATCH] doc/patchwork: Document the patchwork states
michael at niedermayer.cc
Wed Oct 26 20:31:38 EEST 2016
On Sat, Oct 22, 2016 at 09:20:20PM +0200, Michael Niedermayer wrote:
> On Sat, Oct 22, 2016 at 10:41:29AM -0800, Lou Logan wrote:
> > 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".
> Removed "Under Review" and "Awaiting Upstream"
> Theres a patch with "Deferred" state, this would need to be changed
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel