|
From: | Ben Hilburn |
Subject: | Re: [Discuss-gnuradio] Bug in digital::glfsr_source_b |
Date: | Thu, 6 Oct 2016 12:17:02 -0400 |
Hi Eugene,
that sounds like a bug, and a fix, indeed!
I'd immediately go ahead and just replace the int by uint32_t (which happens to be the thing we want to use here, considering the degree self-limits to 32).
But: I know this has been around 0.75 eterneties; Michael even fixed some compiler warning in glfsr.cc last year; I want to be sure we're not breaking anything hence I'm, hereby asking around:
was there a rationale behind having signed intage here?
Best regards,
Marcus
On 09.08.2016 02:29, Eugene Grayver wrote:
Note: It is, of course, possible to trick the blocks into working by passing in negative values (n - 2**32), but it is not elegant.
________________________
Eugene Grayver, Ph.D.
Aerospace Corp., Sr. Eng. Spec.
Tel: 310.336.1274
________________________
From: Eugene Grayver
Sent: Monday, August 8, 2016 5:22 PM
To: address@hidden
Subject: Re: Bug in digital::glfsr_source_bNote: same issue with descrambler_bb and scrambler_bb. The digital.lfsr block is correct.
________________________
Eugene Grayver, Ph.D.
Aerospace Corp., Sr. Eng. Spec.
Tel: 310.336.1274
________________________
From: Eugene Grayver
Sent: Monday, August 8, 2016 2:47 PM
To: address@hidden
Subject: Bug in digital::glfsr_source_bI found a minor (?) bug in glfsr_source_b and _f, and in glfsr.h. The input types for mask and seed should be 'unsigned int', not 'int'. Currently it does not allow 32-bit values, only 31-bit values. While somebody is fixing that, they should also make 'length' an optional input to have the LFSR restart after fewer than 2^degree outputs.
________________________
Eugene Grayver, Ph.D.
Aerospace Corp., Sr. Eng. Spec.
Tel: 310.336.1274
________________________
_______________________________________________ Discuss-gnuradio mailing list address@hidden https://lists.gnu.org/mailman/ listinfo/discuss-gnuradio
_______________________________________________
Discuss-gnuradio mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/discuss-gnuradio
[Prev in Thread] | Current Thread | [Next in Thread] |