[FFmpeg-devel] [PATCH] Documentation: -vf colormatrix options

Katherine Frances Nagels kfnagels at gmail.com
Mon Mar 13 01:00:39 EET 2017


Apologies - now with more useful commit message.
I actually remade the commit since I realized I wasn't connected to my
usual git login. I hope you can kill the other commit. Sorry for this. :(

Contents as before:

>From 99c2763fe7e455bf1d5acd68ae7080509da89ce1 Mon Sep 17 00:00:00 2001
From: kfrn <knfrances at gmail.com>
Date: Mon, 13 Mar 2017 11:57:11 +1300
Subject: [PATCH] doc/filters: Add colourspace values for colormatrix filter

---
 doc/filters.texi | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/doc/filters.texi b/doc/filters.texi
index 192a81a..950ff81 100644
--- a/doc/filters.texi
+++ b/doc/filters.texi
@@ -5260,15 +5260,24 @@ The accepted values are:
 @item bt709
 BT.709

+ at item fcc
+FCC
+
 @item bt601
 BT.601

+ at item bt470
+BT.470
+
+ at item bt470bg
+BT.470BG
+
+ at item smpte170m
+SMPTE-170M
+
 @item smpte240m
 SMPTE-240M

- at item fcc
-FCC
-
 @item bt2020
 BT.2020
 @end table
-- 
2.7.4



On Mon, Mar 13, 2017 at 9:29 AM, Michael Niedermayer <michael at niedermayer.cc
> wrote:

> On Sun, Mar 12, 2017 at 09:49:37AM +1300, Katherine Frances Nagels wrote:
> > Hi ffmpeg-devel,
> >
> > This is my first time submitting a patch, so I hope I'm on the right
> track.
> >
> > I noticed that not all of the -vf colormatrix options described in ffmpeg
> > -h filter=colormatrix are included in the docs at
> > https://ffmpeg.org/ffmpeg-all.html#colormatrix.
> > Therefore, I've added the missing options.
> >
> > I've attached the patch (via git format-patch -1), here are the contents
> > directly:
> >
> > From 0bdaddb3ab2d96a975ac29307ecd2bd7a3ab177c Mon Sep 17 00:00:00 2001
> > > From: Katherine Nagels <kfnagels at gmail.com>
> > > Date: Sun, 12 Mar 2017 09:39:47 +1300
> > > Subject: [PATCH] Update colormatrix values ---
> > > doc/filters.texi | 15 ++++++++++++---
> > > 1 file changed, 12 insertions(+), 3 deletions(-) diff --git
> > > a/doc/filters.texi b/doc/filters.texi
> > > index 192a81a..950ff81 100644
> > > --- a/doc/filters.texi
> > > +++ b/doc/filters.texi
> > > @@ -5260,15 +5260,24 @@ The accepted values are:
> > > @item bt709
> > > BT.709
> > >
> > > + at item fcc
> > > +FCC
> > > +
> > > @item bt601
> > > BT.601
> > >
> > > + at item bt470
> > > +BT.470
> > > +
> > > + at item bt470bg
> > > +BT.470BG
> > > +
> > > + at item smpte170m
> > > +SMPTE-170M
> > > +
> > > @item smpte240m
> > > SMPTE-240M
> > >
> > > - at item fcc
> > > -FCC
> > > -
> > > @item bt2020
> > > BT.2020
> > > @end table
> > > --
> > > 2.7.4
> >
> >
> > Thank you.
> >
> > Best, Katherine
>
> >  filters.texi |   15 ++++++++++++---
> >  1 file changed, 12 insertions(+), 3 deletions(-)
> > 836420957f84fcf3b9c9253d2419d632520b05c2  0001-Update-colormatrix-
> values.patch
> > From 0bdaddb3ab2d96a975ac29307ecd2bd7a3ab177c Mon Sep 17 00:00:00 2001
> > From: Katherine Nagels <kfnagels at gmail.com>
> > Date: Sun, 12 Mar 2017 09:39:47 +1300
>
> > Subject: [PATCH] Update colormatrix values
>
> This commit message does not comply to to our formating requiremets
> its also too general. "update" can mean alot
>
>
> [...]
>
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> No snowflake in an avalanche ever feels responsible. -- Voltaire
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-doc-filters-Add-colourspace-values-for-colormatrix-f.patch
Type: text/x-patch
Size: 727 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170313/e4616118/attachment.bin>


More information about the ffmpeg-devel mailing list