[FFmpeg-devel] [PATCH 2/5] avformat/imf: CPL processor

Pierre-Anthony Lemieux pal at sandflow.com
Wed Oct 6 03:45:35 EEST 2021


On Tue, Oct 5, 2021 at 5:03 PM Lynne <dev at lynne.ee> wrote:
>
> 5 Oct 2021, 17:22 by pal at sandflow.com:
>
> > Hi Lynne,
> >
> > Can you be more specific?
> >
> > The file header we used look consistent with the header of other
> > BSD-licensed files, like aacdec.c.
> >
> > Thanks,
> >
> > -- Pierre
> >
> > On Thu, Sep 30, 2021 at 6:40 AM Lynne <dev at lynne.ee> wrote:
> >
> >>
> >> 29 Sept 2021, 22:47 by pal at sandflow.com:
> >>
> >> > From: Pierre-Anthony Lemieux <pal at sandflow.com>
> >> >
> >> > Signed-off-by: Pierre-Anthony Lemieux <pal at sandflow.com>
> >> > ---
> >> >
> >> > Notes:
> >> >  Implements IMF Composition Playlist (CPL) parsing.
> >> >
> >> >  libavformat/imf_cpl.c | 652 ++++++++++++++++++++++++++++++++++++++++++
> >> >  1 file changed, 652 insertions(+)
> >> >  create mode 100644 libavformat/imf_cpl.c
> >> >
> >> > diff --git a/libavformat/imf_cpl.c b/libavformat/imf_cpl.c
> >> > new file mode 100644
> >> > index 0000000000..e0c8e2c58f
> >> > --- /dev/null
> >> > +++ b/libavformat/imf_cpl.c
> >> > @@ -0,0 +1,652 @@
> >> > +/*
> >> > + * This file is part of FFmpeg.
> >> > + *
> >> > + * Copyright (c) Sandflow Consulting LLC
> >> > + *
> >> > + * Redistribution and use in source and binary forms, with or without
> >> > + * modification, are permitted provided that the following conditions are met:
> >> > + *
> >> > + * * Redistributions of source code must retain the above copyright notice, this
> >> > + *   list of conditions and the following disclaimer.
> >> > + * * Redistributions in binary form must reproduce the above copyright notice,
> >> > + *   this list of conditions and the following disclaimer in the documentation
> >> > + *   and/or other materials provided with the distribution.
> >> > + *
> >> > + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
> >> > + * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
> >> > + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
> >> > + * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
> >> > + * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
> >> > + * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
> >> > + * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
> >> > + * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
> >> > + * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
> >> > + * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
> >> > + * POSSIBILITY OF SUCH DAMAGE.
> >> > + */
> >> >
> >>
> >> Please use the standard file headers we use.
> >>
>
> Please don't top-post.
> libavcodec/aacdec.c's header isn't BSD. It's LGPL, like most of our code.

Oh. I think I meant libavcodec/aadec.c.

> We have a test that runs to check the header of every file in our repo,

Do you mean `make fate-source`? If so, the patch currently passes the test.

> and unfortunately, that test doesn't have a law degree. So, you have to copy
> the header verbatim from other files (which you can BSD is compatible
> with LGPL, so if you copy the author's attributions, you can) or add an exception

The intent is not to add an exception, just offer the code under
2-clause BSD. Ok?

> (we generally don't, except for some optional GPL code we have).
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list