[FFmpeg-devel] [PATCH 1/4] dxva2: make ff_dxva2_get_surface() static and rename it

[Michael Niedermayer]

On Fri, Jan 06, 2017 at 09:52:48AM +1100, Hendrik Leppkes wrote:
> On Fri, Jan 6, 2017 at 9:15 AM, Michael Niedermayer
> <michael at niedermayer.cc> wrote:
> > On Wed, Jan 04, 2017 at 02:36:41PM +0100, Steve Lhomme wrote:
> >> ---
> >>  libavcodec/dxva2.c          | 8 ++++----
> >>  libavcodec/dxva2_internal.h | 2 --
> >>  2 files changed, 4 insertions(+), 6 deletions(-)
> >
> > applied
> >
> > PS: it seems this was posted over a month ago with no reviewes or
> > comments, is some co-maintainer needed for this ?
> >
> > thx
> >
> 
> This set was being reviewed on the libav mailing list, it makes no
> sense to review it twice.

It makes sense to approve or reject patches. Otherwise noone knows
and also patchwork has the status wrong then.

Furthermore without approval, patches wont neccessarily get applied
and without rejection patches may get applied that someone would have
arguments against.

So i think the maintainer should reply to patches here with either
a review, an approval or a rejection or something along these lines.
And in that light, was my question about a co-maintainer ...

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170106/4446e293/attachment.sig>