Re: [PATCH 2/2] Add a simple test for LFSR generator

[Alex Pyrgiotis <apyrgio@grnet.gr>]

On 03/11/2013 05:51 PM, Jiri Horky wrote:
> Hi Alex,
>
> I just used your implementation in a different project as well, thanks 
> for it :) One small correction, I think that the last line of:
>
> On 03/08/2013 01:37 PM, Alex Pyrgiotis wrote:
>> +    /* Check if all expected numbers within range have been 
>> calculated */
>> +    r = 0;
>> +    if (verify) {
>> +        fprintf(stderr, "Verifying results... ");
>> +        for (i = 0; i < numbers; i++) {
>> +            if (*(uint8_t *)(v + 1) != 1) {
> should be:
>
>             if (*(uint8_t *)(v + i) != 1) {
>
> (not +1, but +i).
>
>

Hi Jiri,

I'm glad you found the implementation useful.
Your observation is correct. I'll send a new patch that fixes this 
issue, among other things.

Thanks,
Alex

-- 
Alex | apyrgio@grnet.gr