[FFmpeg-devel] [PATCH] libavutil/doc: Changes in documentation due to changes to fixed_dsp

[wm4]

On Fri, 5 Jun 2015 20:05:33 +0200
Michael Niedermayer <michaelni at gmx.at> wrote:

> On Fri, Jun 05, 2015 at 07:59:47PM +0200, Michael Niedermayer wrote:
> > On Fri, Jun 05, 2015 at 10:04:54AM -0400, Ronald S. Bultje wrote:
> > > Hi,
> > > 
> > > On Fri, Jun 5, 2015 at 5:54 AM, Nedeljko Babic <nedeljko.babic at imgtec.com>
> > > wrote:
> > > 
> > > > New functions are added to fixed_dsp, so the documentation is changed
> > > > accordingly.
> > > >
> > > > Signed-off-by: Nedeljko Babic <nedeljko.babic at imgtec.com>
> > > > ---
> > > >  doc/APIchanges      | 7 +++++++
> > > >  libavutil/version.h | 2 +-
> > > >  2 files changed, 8 insertions(+), 1 deletion(-)
> > > >
> > > > diff --git a/doc/APIchanges b/doc/APIchanges
> > > > index 5c36dca..bcf4fe6 100644
> > > > --- a/doc/APIchanges
> > > > +++ b/doc/APIchanges
> > > > @@ -15,6 +15,13 @@ libavutil:     2014-08-09
> > > >
> > > >  API changes, most recent first:
> > > >
> > > > +2015-06-05 - xxxxxxx - lavu  54.27.101 - fixed_dsp.h
> > > > +  Add vector_fmul()
> > > > +  Add vector_fmul_reverse()
> > > > +  Add vector_fmul_add()
> > > > +  Add scalarproduct_fixed()
> > > > +  Add butterflies_fixed()
> > > 
> > > 
> > > These functions are not part of our public API, are they?
> > 
> > It appears that the header isnt installed currently, ill post a patch
> > to correct that
> 
> hmm, it seems thats not possible
> it contains
> #include "libavcodec/mathops.h"
> which is not a public header and from libavcodec
> 
> ill revert the addition to APIchanges
> 
> nedeljko, can you fix that header so it can be installed or move the
> content to libavcodec. Depending on what the people prefer

I'd prefer not adding new public API for no reason.