[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Discuss-gnuradio] digital_correlate_access_code_bb
From: |
Tom Rondeau |
Subject: |
Re: [Discuss-gnuradio] digital_correlate_access_code_bb |
Date: |
Mon, 9 Apr 2012 13:38:22 -0400 |
On Sat, Apr 7, 2012 at 10:12 PM, Marcus D. Leech <address@hidden> wrote:
> Just looking at this function:
>
> correlate_access_code_bb
>
> In the method set_access_code, it takes a string. Which should be ASCII '1'
> and '0' characters to represent the binary sequence being
> correlated against.
>
> Here's a little beauty of a code snippet:
>
> d_access_code = 0;
> for (unsigned i=0; i < 64; i++){
> d_access_code <<= 1;
> if (i < len)
> d_access_code |= access_code[i] & 1; // look at LSB only
> }
>
> This relies on the fact that ASCII '1' and '0' happen to have low-order bits
> of the right "flavour". This is insanely dirty and gross and I can't
> believe we let this nonsense in the code base.
>
> There's no reason not to do the right thing here.
>
>
> --
> Marcus Leech
> Principal Investigator
> Shirleys Bay Radio Astronomy Consortium
> http://www.sbrac.org
Want to submit a patch?
Tom
- [Discuss-gnuradio] digital_correlate_access_code_bb, Marcus D. Leech, 2012/04/07
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Thomas Tsou, 2012/04/07
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb,
Tom Rondeau <=
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Marcus D. Leech, 2012/04/09
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Nick Foster, 2012/04/09
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Tom Rondeau, 2012/04/09
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Nowlan, Sean, 2012/04/15
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Tom Rondeau, 2012/04/19
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Nowlan, Sean, 2012/04/19
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Tom Rondeau, 2012/04/23
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Nowlan, Sean, 2012/04/23
- Re: [Discuss-gnuradio] digital_correlate_access_code_bb, Tom Rondeau, 2012/04/23