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

Hendrik Leppkes h.leppkes at gmail.com
Fri Jan 6 05:55:05 EET 2017


On Fri, Jan 6, 2017 at 11:11 AM, Michael Niedermayer
<michael at niedermayer.cc> wrote:
> 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 ...

While review is still on-going, there is no such status to apply.

- Hendrik


More information about the ffmpeg-devel mailing list