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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxx

--
To unsubscribe from this list: send the line "unsubscribe fio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel]     [Linux SCSI]     [Linux IDE]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux