[Ffmpeg-devel] thp audio decoder

Michael Niedermayer michaelni
Wed Apr 4 11:41:44 CEST 2007


Hi

On Wed, Apr 04, 2007 at 12:34:05PM +0400, Kislyakov Maxim wrote:
> Hi,
> as i see i have an error either in my system or compiler, maybe it is
> too old and needed to be upgraded to fix errors, it works, but to make
> any changes work i need firstly to clean than to recompile full
> project, it uses much time on my PC, so i couldn't properly debug the
> code.
> I overlooked the patch it seems to work, but to track errors is
> difficult. Here is the diff from 2007-03-25 snapshot and thp.c file.

[...]
> diff -br snapshot/libavcodec/adpcm.c ffmpeg/libavcodec/adpcm.c
> 154a155,164
> > typedef struct ADPCMThpFrameHeader {
> >     uint32_t channelSize;
> >     uint32_t numSamples;
> >     float tableFC[16];
> >     float tableSC[16];
> >     int16_t channel1Prev1;
> >     int16_t channel1Prev2; 
> >     int16_t channel2Prev1;
> >     int16_t channel2Prev2;
> > } ADPCMThpFrameHeader;
> 774a785
> >         
> 804a816,846
> > 
> > static inline short adpcm_thp_expand_nibble(ADPCMThpFrameHeader *frameHeader, int8_t nibble, 
> > 					    int8_t firstByte, int st)

please send a proper single unified diff against svn head of your changes
like everyone else!
also loose the tabs and trailing whitespace and use 4 space indention

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

There will always be a question for which you do not know the correct awnser.
-------------- 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/20070404/34f2b2dd/attachment.pgp>



More information about the ffmpeg-devel mailing list