2015-01-27 22:01 GMT+01:00 supraja reddy <supraja0...@gmail.com>:
> Hello,
>
>> + for (i = 0; i < cs->ksize; i++) {
>> > + Me[i] = Key[2 * i];
>> > + Mo[i] = Key[2 * i + 1];
>> > + }
>> > + for (i = 0; i < cs->ksize; i++)
>> > + cs->S[cs->ksize - i - 1] = tf_RS(Me[i], Mo[i]);
>>
>> Can't the previous two for loops be merged together?
>>
>>
> You wanted me to divide the two sets earlier so I had split them up. I have
> merged them now anyways.
OK :) , at least you're using only one variable for the loop (i) now
and not two (i, j).
>
>
>> > +/**
>> > + * Initialize an AVTWOFISH context.
>> > + *
>> > + * @param ctx an AVTWOFISH context
>> > + * @param key a key of size ranging from 1 to 32 bytes used for
>> encryption/decryption
>> > + * @param key_bits number of keybits: 128, 192, 256 If less than the
>> required, padded with zeroes to nearest valid value
>> > + */
>>
>> You need to document the return values of this function.
>>
>
> I hope the documentation is proper and as requested.
I meant document the return value using @return, but we can fix it
after we apply the patch.
The rest LGTM, thanks.
Giorgio Vazzana
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
