[FFmpeg-devel] [PATCH] MAINTAINERS: add twitter entry
Timothy Gu
timothygu99 at gmail.com
Sat Oct 5 02:56:00 CEST 2013
On Oct 4, 2013 5:43 PM, "Lou Logan" <lou at lrcd.com> wrote:
>
> On Fri, 4 Oct 2013 17:11:51 -0700, Timothy Gu wrote:
>
> > On Oct 4, 2013 3:27 PM, "Lou Logan" <lou at lrcd.com> wrote:
> > >
> > > Add myself (Lou Logan) as twitter maintainer.
> > > https://twitter.com/FFmpeg
> > > ---
> > > MAINTAINERS | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 53ef452..df3e771 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -51,7 +51,7 @@ mailinglists Michael
> > Niedermayer, Baptiste Coudurier,
> > > presets Robert Swain
> > > metadata subsystem Aurelien Jacobs
> > > release management Michael Niedermayer
> > > -
> > > +twitter Lou Logan
> >
> > Nit: Twitter capitalize
>
> I was following the lowercase convention of the other entries.
> Capitalized or not is fine with me, but I'm not going to make an
> updated patch to do so.
Twitter® is a company/product name, and thus should be capitalized.
>
> > >
> > > libavutil
> > > =========
> > > --
> > > 1.8.4
> >
> > LGTM. Also it would be better if have a separate section containing all
the
> > social stuff, like website, Twitter, and Google +.
>
> Fine with me, but this can be in separate commit(s)
Of course.
> if or when someone
> adds the g+ info, and I'm not sure who the g+ overlord(s) are.
I am not sure either. (Paul?)
> However,
> I will prefer the header "Communication" instead of "Social".
+1. I couldn't find the right word for that when I was writing the mail.
(Maybe that's the difference between having native and having semi-native
English?)
Timothy
More information about the ffmpeg-devel
mailing list