[MPlayer-dev-eng] [PATCH] de-templatize TV new_handle function
Diego Biurrun
diego at biurrun.de
Mon Sep 13 20:00:22 CEST 2010
On Mon, Sep 13, 2010 at 07:49:15PM +0200, Reimar Döffinger wrote:
> On Mon, Sep 13, 2010 at 06:50:53PM +0200, Diego Biurrun wrote:
> > On Mon, Sep 13, 2010 at 06:36:11PM +0200, Reimar Döffinger wrote:
> > > On Mon, Sep 13, 2010 at 04:58:36PM +0200, Diego Biurrun wrote:
> > > > On Sun, Sep 12, 2010 at 04:06:04PM +0200, Reimar Döffinger wrote:
> > > > > On Sun, Sep 12, 2010 at 01:52:03PM +0200, Diego Biurrun wrote:
> > > > > > +tvi_handle_t *tv_new_handle(const size_t size,
> > > > >
> > > > > What's that const supposed to be good for?
> > > >
> > > > The function parameter is not changed so I figured it would not hurt..
> > >
> > > It also doesn't have any advantage, and it does hurt in so far that
> > > it makes changing the function more annoying (e.g. if you wanted to enforce
> > > a minimum "size" you'd either have to add another variable or remove that
> > > const).
> >
> > New patch without const.
>
> Feel free to just commit whatever you want, but I do have a few "advisory" comments.
I'm happy to hear them, that's why I send patches..
Diego
More information about the MPlayer-dev-eng
mailing list