annotate DOCS/tech/codec-devel.txt @ 19382:7c6c205b88b6

trying to fix the reverting paragraph if you dissagree, dont hesitate to revert this commit or flame, but at least we should not claim that svn cannot revert commits except by recommiting the old version
author michael
date Sun, 13 Aug 2006 22:14:32 +0000
parents 56bb20a00bc9
children 3eeaf9d4c65a
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
3862
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
1 A Guide To Developing MPlayer Codecs
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
2 by Mike Melanson (melanson at pcisys dot net)
7349
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
3 updated to libmpcodecs arch by A'rpi
3862
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
4
7399
56bb20a00bc9 cross-ref
arpi
parents: 7349
diff changeset
5 SEE ALSO: libmpcodecs.txt !!!
56bb20a00bc9 cross-ref
arpi
parents: 7349
diff changeset
6
3862
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
7 Introduction
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
8 ------------
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
9 I've developed a number of open source decoders for the MPlayer project,
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
10 for both audio and video data. As such, I feel I'm qualified to document a
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
11 few notes about developing new codecs for the codebase.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
12
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
13 As always, the best way to learn how to incorporate a new codec is to
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
14 study a bunch of existing code. This document is supplementary material to
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
15 the code, meant to give some tips, pointers, and a general roadmap.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
16
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
17 A note about terminology: "Codec" stands for coder/decoder (or
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
18 compressor/decompressor, if you prefer). The term refers to a module that
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
19 can both encode and decode data. However, this document focuses primarily
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
20 on incorporating decoders. Still, the terms "decoder" and "codec" are
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
21 often used interchangeably.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
22
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
23 Necessary Materials
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
24 -------------------
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
25 So you've decided that you want to implement a new decoder for
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
26 MPlayer. There are a few things you will need:
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
27
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
28 - Knowledge of the codec to be implemented: You will need to know the data
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
29 format of the chunks that MPlayer will pass to you. You will need to know
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
30 how to take apart the data structures inside. You will need to know the
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
31 algorithmic operations that need to be performed on the data in order to
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
32 reconstruct the original media.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
33
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
34 - Sample media: Preferably, lots of it. You will need media encoded in
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
35 your data format and stored in a media file format that MPlayer knows how
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
36 to parse (these include AVI, ASF, MOV, RM, VIVO, among others). If the
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
37 encoded data is stored in a media file format that MPlayer doesn't
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
38 understand, then you will either need to somehow convert the format to a
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
39 media file format that the program does understand, or write your own
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
40 MPlayer file demuxer that can handle the data. Writing a file demuxer
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
41 is beyond the scope of this document.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
42 Try to obtain media that stresses all possible modes of a
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
43 decoder. If an audio codec is known to work with both mono and stereo
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
44 data, search for sample media of both types. If a video codec is known to
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
45 work at 7 different bit depths, then, as painful as it may be, do what you
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
46 can to obtain sample media encoded for each of the 7 bit depths.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
47
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
48 - Latest CVS snapshot: It's always useful to develop code for the very
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
49 latest development version of MPlayer. Be sure to update your local CVS
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
50 copy often.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
51
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
52 - General programming knowledge, working Linux development environment: I
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
53 would hope that these items would go without saying, but you never know.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
54
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
55 Typical Development Cycle
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
56 -------------------------
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
57 1) Set up basic infrastructure
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
58 First things first, there's a big song and dance to go through in order to
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
59 let the MPlayer program know that you have a new codec to incorporate.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
60
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
61 First, modify your local copy of codecs.conf. It may be system-shared or
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
62 in your home directory. Add a new entry for your codec. If it's an open
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
63 source codec, it would be a good idea to place the new entry with the rest
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
64 of the open source codecs. When you're confident that you have the entry
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
65 right, be sure to add it to etc/codecs.conf in your workspace. See the
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
66 file codecs.conf.txt for a detailed description of the format of this
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
67 file. Create a new audiocodec or videocodec block with the proper info,
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
68 FOURCCs/format numbers, output formats, and a unique driver name. Remember
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
69 the driver name.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
70
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
71 Next, create a new source file which contains the main decoding function
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
72 that MPlayer will call to decode data. Eventually, you may have multiple
7349
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
73 files which comprise your decoder, but let's start simple here.
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
74 For audio codecs, see ad_sample.c skeleton. For video, choose one of the
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
75 existing vd_*.c files which you think is close to your codec in behaviour.
3862
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
76
7349
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
77 Next, modify the Makefile so that it will compile your new source file.
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
78 Also, add your codec to the array in ad.c (for audio) or vd.c (for video).
3862
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
79
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
80 Next, compile the project and see if you have everything correct so far.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
81
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
82 Next, you want to make sure that the encoded data is making it to your
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
83 decoding function in the first place. This may sound like a trivial
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
84 exercise, but there are a lot of things that can go wrong (and I've
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
85 watched most of them go wrong in my experience). At the beginning of your
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
86 skeleton decoder function, enter the following code:
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
87 int i;
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
88 for (i = 0; i < 16; i++)
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
89 printf ("%02X ", input[i]);
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
90 printf ("\n");
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
91 When you compile and run MPlayer, your decoder function will print the
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
92 first 16 bytes of each data chunk that it receives. Open the sample media
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
93 in a hex editor and reconcile what you see on the screen with what
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
94 you find in the file. If the decoder is printing the first 16 bytes of
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
95 each block, that's a good sign that you're ready to move on to step
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
96 2. Otherwise, you need to figure out why the data isn't getting to your
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
97 decoder. Is your decoder even being invoked? If not, why not?
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
98
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
99 2) Develop the decoder
4630
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
100 Go for it. Remember to make it work, first, then make it work fast. Some
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
101 specific tips:
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
102
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
103 What output formats should you support in your decoder? Whatever makes
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
104 sense. YUV output is always preferable over RGB output. Generally, if a
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
105 codec uses a YUV data as its source data, you will be able to decode a
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
106 frame of YUV data. If a codec takes RGB data as its input, as many older
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
107 video codecs do, then there's no point in supporting YUV output; just
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
108 output as many RGB formats as possible.
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
109
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
110 The most preferred output format for video data is YV12. This is because
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
111 MPlayer supports a multitude of hardware devices that can display, scale,
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
112 and filter this type of data directly. MPlayer also has a bunch of
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
113 optimized conversion functions that can convert YV12 data to any other
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
114 type of output data.
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
115
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
116 If you do take the RGB output route, you should be aware that MPlayer
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
117 actually orders packed RGB data as BGR. If you're decoding into a BGR24
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
118 buffer, the output will look like:
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
119 B G R B G R B G R B ...
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
120 If you're decoding into a BGR32 buffer, there will need to be an
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
121 additional (unused) byte after each BGR triplet:
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
122 B G R - B G R - B G ...
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
123
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
124 Make liberal use of sanity checks. Start by including the file mp_msg.h at
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
125 the start of your decoder. Then you can use the mp_msg() function as you
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
126 would a normal printf() statement. Whenever your decoder notices a strange
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
127 bit of data or an odd condition, print a message such as:
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
128 mp_msg(MSGT_DECVIDEO, MSGL_WARN, "Odd data encountered: %d\n", data);
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
129 Obviously, you should make the message a little more
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
130 descriptive, for your benefit. MSGL_WARN is a good message level for this
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
131 type of information. Look in mp_msg.h for all of the error levels. You can
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
132 even make MPlayer bail out completely by using MSGL_FATAL, but that should
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
133 never be necessary at the data decoder level.
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
134
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
135 What conditions should trigger a warning? Anything, and I mean *anything*
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
136 out of the ordinary. Many chunks of compressed video data contain headers
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
137 with data such as width, height, and chunk size. Reconcile these fields
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
138 with the parameters passed into the decoding function (if you set it up to
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
139 take those parameters). Such data should match up. If it doesn't, issue a
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
140 warning and make an executive decision in the code about which data to
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
141 believe (personally, I always lend more weight to the data that was passed
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
142 into the decoder function, the data that comes from the container file's
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
143 header). If there's supposed to be a magic number embedded in, or computed
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
144 from, the chunk's header, issue a warning if it isn't correct.
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
145
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
146 Whenever you're about the index into a memory array with an index that
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
147 could theoretically be out of range, then test that the index is in range,
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
148 no matter how tedious it seems. Accessing outside of your memory range is,
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
149 after all, the number 1 cause of segmentation faults. Never trust that all
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
150 the data passed to you will be correct. If an array index suddenly winds
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
151 up out of range, it's probably best to issue a warning about it and bail
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
152 out of the decoder (but not the whole application).
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
153
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
154 Writing all of these warning statements may seem insipid, but consider
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
155 that if you don't do it when you start writing your decoder, you'll
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
156 probably end up doing it later on when your decoder isn't working properly
02b275e198c3 added more tips on codec development
melanson
parents: 3862
diff changeset
157 and you need to figure out why (believe me, I know).
3862
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
158
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
159 3) Debug and test the decoder
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
160 If you're extremely lucky, the decoder will work the first time. If you're
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
161 very lucky, it will work after you've reviewed your code a few times and
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
162 corrected a few obvious programming mistakes. Realistically, you will
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
163 write the decoder, review it many times and fix many obvious and subtle
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
164 programming errors, and still have to go through an elaborate debug
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
165 process in order to get the decoder to a minimally functional state.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
166
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
167 Big hint: Ask for all warnings. You know, the -Wall option in
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
168 gcc? It's very useful to develop your codec while running in debug
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
169 mode. In order to compile MPlayer with debug support (which includes -Wall
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
170 for all gcc operations), use the --enable-debug option when configuring
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
171 the project. Pay attention to all warnings and make it a goal to get
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
172 rid of every single one. I'll never forget when the compiler warned me
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
173 that there was no point in clamping a signed 16-bit variable within a
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
174 signed 16-bit range (the calculation to be clamped was supposed to be
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
175 stored in a signed 32-bit variable and then stored in the signed 16-bit
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
176 variable). I sat stunned for a moment, feeling like I had just dodged a
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
177 bullet as I knew that would have taken me hours to debug that kind of
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
178 mistake.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
179
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
180 4) Contribute decoder to codebase
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
181 Create a patch with the "diff -u" format and email it to the MPlayer
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
182 development team for approval. You will likely need to diff the following
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
183 files:
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
184 - Makefile
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
185 - etc/codecs.conf
7349
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
186 - ad.c or vd.c
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
187 Of course, you will need to include your newly-created file(s):
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
188 vd_<name>.c -OR- ad_<name>.c. If you contribute enough decoders, the
c5fd28225fab updated to libmpcodecs way
arpi
parents: 4630
diff changeset
189 development team may even grant you write privileges to the CVS repository.
3862
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
190
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
191 5) Wait for bug reports to start rolling in
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
192 You may think you're finished when you release the codec and if you're
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
193 extremely lucky, you will be right. However, it's more likely that people
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
194 will start throwing all kinds of oddball media at your decoder that it
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
195 never counted on. Cheer up; take comfort in knowing that people are
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
196 testing your code and attempting to use it as a real world
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
197 application. Download the problem media that people upload to the MPlayer
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
198 FTP site and get back to work, implementing fixed code that addresses the
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
199 issues. Contribute more patches and encourage people to hammer on your
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
200 decoder even more. This is how you make your decoder rock-solid.
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
201
74622f23bce5 initial commit of "Guide To Codec Hacking"
melanson
parents:
diff changeset
202 EOF