[FFmpeg-devel] [PATCH] adpcm: Clarify a comment

Michael Niedermayer michaelni
Wed Nov 3 20:21:59 CET 2010


On Wed, Nov 03, 2010 at 09:07:26PM +0200, Martin Storsjo wrote:
> ---
>  libavcodec/adpcm.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c
> index 455b477..f298ef5 100644
> --- a/libavcodec/adpcm.c
> +++ b/libavcodec/adpcm.c
> @@ -354,7 +354,7 @@ static void adpcm_compress_trellis(AVCodecContext *avctx, const short *samples,
>          int sample = samples[i*stride];
>          memset(nodes_next, 0, frontier*sizeof(TrellisNode*));
>          for(j=0; j<frontier && nodes[j]; j++) {
> -            // higher j have higher ssd already, so they're unlikely to use a suboptimal next sample too
> +            // higher j have higher ssd already, so they're likely to yield a suboptimal next sample too

how much is gained without that assumtion?

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

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101103/504db816/attachment.pgp>



More information about the ffmpeg-devel mailing list