[FFmpeg-devel] [PATCH v14 9/9] avcodec/evc: Changes in Changelog and MAINTAINERS files

[Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics]

-----Original Message-----
From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of Michael
Niedermayer
Sent: środa, 14 grudnia 2022 22:36
To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH v14 9/9] avcodec/evc: Changes in
Changelog and MAINTAINERS files

On Tue, Dec 13, 2022 at 08:33:29AM -0500, Ronald S. Bultje wrote:
> Hi David,
> 
> On Tue, Dec 13, 2022 at 7:22 AM Dawid Kozinski/Multimedia (PLT) 
> /SRPOL/Staff Engineer/Samsung Electronics <d.kozinski at samsung.com> wrote:
> 
> > Should I leave the following lines:
> > +  libxevd.c                             Dawid Kozinski
> > +  libxeve.c,                            Dawid Kozinski
> > +  evc.c, evc.h                        Dawid Kozinski
> > +  evcdec.c                             Dawid Kozinski
> > +  evc_parser.c                      Dawid Kozinski
> >
> > or should I remove them?
> >
> 
> Here's a question for you, and the answer probably becomes 
> self-evident from that. If you, Dawid, stop working for Samsung, for 
> example because you're starting your own business or Samsung fires you 
> or Google hires you, or if Samsung stops sponsoring this new codec 
> called "EVC" or stops contributing to this new library "libxeve". Will 
> you, Dawid, still maintain these files?
> 
> If the answer is yes, then you can shorten these lines ("evc*.[ch]: Dawid"
> & "livxev[ed].c: Dawid") and keep them.
> 
> If the answer is no, then I think you should remove (or adjust) these 
> lines, since they are (in their current form) inaccurate: you are not 
> maintaining these files, your company is.
> 

I think for code maintained by a company we still should list a person
because persons can be contacted while large companies are sometimes very
difficult to contact.
maybe
Dawid Kozinski (Samsung) or Samsung (Dawid Kozinski) or something like that
would specify this better

thx

DK
Since there is no further discussion related to the MAINTAINERS file, I've
added "Samsung (Dawid Koziński)" entries next to the EVC codec related
files.

I've just submitted the newest version of our implementation and waiting for
the review or merge.

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

"I am not trying to be anyone's saviour, I'm trying to think about the
future and not be sad" - Elon Musk