[FFmpeg-devel] [PATCH] MAINTAINERS: add twitter entry

[Timothy Gu]

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