[Ffmpeg-devel] #include dsputil.h needed in opt.c

Diego Biurrun diego
Tue Sep 20 01:25:22 CEST 2005


On Mon, Sep 19, 2005 at 08:17:52PM +0200, Michael Niedermayer wrote:
> 
> On Mon, Sep 19, 2005 at 07:19:50PM +0200, Diego Biurrun wrote:
> > On Mon, Sep 19, 2005 at 11:03:41AM +0200, Michael Niedermayer wrote:
> > > 
> > > On Mon, Sep 19, 2005 at 08:12:39AM +0200, Xavier Rodriguez (INEXBEE) wrote:
> > > > 
> > > > I have seen that lrintf function was pre-processed in the dsputil.h file and
> > > > used by several c files in order to manage the fact that some systems like
> > > > the solaris ones do not have this function in its libraries. The problem is
> > > > that libavcodec/opt.c file uses the lrintf function but does not include
> > > > dsputil.h file.
> > > 
> > > maybe dsputil.h is not the best place for the lrintf() mess, maybe common.h
> > > would be better ...
> > 
> > Would something like the attached patch do?
> 
> yes

Applied.

> > I have a question about how to best handle #indclude: There are several
> > places that include both avcodec.h and common.h.  Now avcodec.h
> > #includes avutil.h, which in turn #includes common.h.  Is it better to
> > #include common.h explicitly or implicitly?
> 
> dunno, but it should be consistently done

I've left them in place for the moment.

Diego





More information about the ffmpeg-devel mailing list