[FFmpeg-devel] [PATCH] Add libavcodec/dpcm.c codec long_names

Kostya kostya.shishkov
Thu May 15 13:36:16 CEST 2008


On Thu, May 15, 2008 at 12:01:04PM +0200, Stefano Sabatini wrote:
> On date Thursday 2008-05-15 11:55:14 +0300, Kostya encoded:
> > On Thu, May 15, 2008 at 09:23:31AM +0200, Stefano Sabatini wrote:
> > > On date Wednesday 2008-05-14 19:09:56 -0700, Mike Melanson encoded:
> > > > Stefano Sabatini wrote:
> > > > > Hi all, as in subject.
> > > > > 
> > > > > By the way, we named in roqaudioenc.c roq_audio_encoder with id
> > > > > CODEC_ID_ROQ_DPCM like "Id RoQ audio", should we change that to "Id
> > > > > RoQ DPCM" for consistency with this patch (or the other way round)?
> > > > 
> > > > Come to think of it, 'id' should be spelling with a lowercase 'i'. And 
> > > > DPCM is more explanatory than audio, yes.
> > > 
> > > OK, so expect another one to change all the instances of Id in id (also
> > > we should update the Multimedia Wiki accordingly).
> > 
> > Err, IIRC they used all combinations of upper/lowercase in their name
> > thoughout the history. Make sure name is historically correct ;) 
> 
> I think "id" is definitively the more correct name, is used in the id
> Software website and it even shows up on the Doom videogame cover back
> in 1993, but it is often written also as "Id" and with less frequency
> as "ID" as you can find out with a google lookout, so I'd say to stick
> with "id" consistently.

The only thing matters is what idlog.cin and idlogo.RoQ display ;)
As they both use "id" it should be so.
 
> Regards.
> -- 
> FFmpeg = Frightening Fast MultiPurpose EnGine




More information about the ffmpeg-devel mailing list