On Tue, Dec 16, 2014 at 03:00:24PM +0530, supraja reddy wrote:
> Hi,
> 
> Attached patch adds some missing spaces in the cast5.c and cast5.h
> 
> Please let me know if any further changes needed.
[...]

> -    for (i=0;i<=cs->rounds;i++)
[...]
> +    for (i = 0; i < cs->rounds; i++)
> +        cs->Kr[i] = cs->Kr[i] & 0x1f;

this is not a cosmetic change and is not correct
Kr[cs->rounds] is used in some shifts
Kr[0] might be unused though

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to