[NUT-devel] [PATCH] fourcc spec

[Oded Shimon]

On Fri, Apr 28, 2006 at 02:10:19PM -0400, Rich Felker wrote:
> On Fri, Apr 28, 2006 at 03:38:17PM +0300, Oded Shimon wrote:
> > On Fri, Apr 28, 2006 at 02:37:41PM +0200, Luca Barbato wrote:
> > > Måns Rullgård wrote:
> > > > Seems sensible.  The problem is only with codecs that don't have an
> > > > official name, like some of Mike's game stuff.
> > > 
> > > they don't have name so won't be a problem having us give them one.
> > > 
> > > > 
> > > 
> > > Oded please comment
> > 
> > I have nothing to comment, I already stated, I'm fine with any suggestion. 
> > (The only suggestion I somewhat dislike is the ugly numbers one)
> > Michael and Rich are the ones that need to comment...
> 
> How about:
> 
> audio:
> "mp2 "
> "mp3 "
> "ac3 "
> "aac "
> vrbs
> 
> video:
> mp4v/mp4s/??? (pick one among those already in use)
> mpg1
> mpg2
> h263
> h264
> 263+ (nonconformant extensions to h263 should not be tagged h263)
> 
> comments? the basic idea of the above is to pick the "most sane" among
> the fourcc's already used in practice in _some_ format out there..

I'm in favor.

- ods15
-------------- next part --------------
length,string

Video
4,"h263"	H.263
4,"mp4s"	MPEG-4 Part 2
4,"h264"	MPEG-4 Part 10/H.264


Audio
4,"vrbs"	Xiph.org Vorbis
4,"mp2 "	MP3
4,"mp3 "	MP2
4,"ac3 "	AC3
4,"aac "	AAC/MPEG-4 Part 3
-------------- next part --------------
Index: DOCS/tech/mpcf.txt
===================================================================
RCS file: /cvsroot/mplayer/main/DOCS/tech/mpcf.txt,v
retrieving revision 1.155
diff -u -r1.155 mpcf.txt
--- DOCS/tech/mpcf.txt	25 Mar 2006 13:39:47 -0000	1.155
+++ DOCS/tech/mpcf.txt	14 Apr 2006 10:33:12 -0000
@@ -439,10 +439,7 @@
           a demuxer MUST ignore streams with reserved classes
 
 fourcc
-    identification for the codec
-    example: "H264"
-    MUST contain 2 or 4 bytes, note, this might be increased in the future
-    if needed
+    identification for the codec, must comply to fourcc.txt
 
 time_base_nom / time_base_denom = time_base
     the length of a timer tick in seconds, this MUST be equal to the 1/fps