[Ffmpeg-devel] THP decoder

Michael Niedermayer michaelni
Sat Apr 7 14:35:07 CEST 2007


Hi

On Sat, Apr 07, 2007 at 04:19:54PM +0400, Kislyakov Maxim wrote:
> >> Hi,
> 
> Hi
> 
> Michael Niedermayer wrote:
> >> Michael Niedermayer wrote:
> >> >> New patch.
> >> > where?
> >> 
> >> I'm sorry, i was half-sleepy yesterday and must have forgotten to
> >> attach it (it was 3 o'clock am). So it is here.
> >> 
> >> Maxim
> >> Index: libavformat/thp.c
> >> ===================================================================
> >> --- libavformat/thp.c   (revision 0)
> >> +++ libavformat/thp.c   (revision 0)
> > [...]
> >> +#include "avformat.h"
> >> +#include "allformats.h"
> >> +
> >> +typedef struct ThpDemuxerContext  {
> >> +    int maxAudioSamples;              /* maximal number of samples in
> >> one frame */
> > 
> > still not doxygen compatible
> 
> I hope fixed

no, they are doxygen comments now but they are still wrong

[...]

> +            else  {
> +                *samples++ = adpcm_thp_expand_nibble(&frameHeader, (src[i] & 0x0f), index, exponent, 0);

superfluous ()

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070407/cb086905/attachment.pgp>



More information about the ffmpeg-devel mailing list