[FFmpeg-devel] [PATCH] Cosmetics: improve long names for ffm muxer and demuxer

[Diego Biurrun]

On Sun, Jul 20, 2008 at 01:58:57PM -0700, Baptiste Coudurier wrote:
> Diego Biurrun wrote:
> > On Sun, Jul 20, 2008 at 11:53:09AM -0700, Baptiste Coudurier wrote:
> >> Diego Biurrun wrote:
> >>> On Sat, Jul 19, 2008 at 06:03:30PM -0700, Baptiste Coudurier wrote:
> >>>> Stefano Sabatini wrote:
> >>>>> --- libavformat/ffmdec.c	(revision 14284)
> >>>>> +++ libavformat/ffmdec.c	(working copy)
> >>>>> @@ -477,7 +477,7 @@
> >>>>>  
> >>>>>  AVInputFormat ffm_demuxer = {
> >>>>>      "ffm",
> >>>>> -    NULL_IF_CONFIG_SMALL("ffm format"),
> >>>>> +    NULL_IF_CONFIG_SMALL("FFserver live feed Muxer"),
> >>>>>      sizeof(FFMContext),
> >>>> I don't think this is the right name.
> >>> Well, it is called by that name in the code:
> >>>
> >>>  * FFM (ffserver live feed) demuxer
> >> Do we add "Muxer" to names ? No, at least not in mxf.c and mov.c
> > 
> > No, we do not.
> 
> So why would you accept this name ?

There is a misunderstanding here.  First, I never meant to imply that
the word "muxer" should be part of the codec long name.  Second, I
understood you to mean that "FFserver live feed" was not the correct
name for the format implemented in ffmdec.c/ffmenc.c.  Is this the
correct name, yes or no?  If no, which is the correct one?

Diego