Re: [Patch-gnuradio] Masking in volk_64u_popcnt.h

[Tom Rondeau]

On Mon, May 7, 2012 at 6:47 PM, Moritz Fischer
<address@hidden> wrote:
> On 05/06/2012 12:43 PM, Moritz Fischer wrote:
>
>> - uint32_t retVal = (uint32_t)(value && 0x00000000FFFFFFFF);
>> + uint32_t retVal = (uint32_t)(value & 0x00000000FFFFFFFF);
>>
>> Patch is attached,
>
>
>
> Sorry for double posting ... but the previously attached patch missed a
> second occurrence of the && in the same file.
>
> Patch attached.
>
> Cheers,
>
> Moritz

Hi Moritz,

Thanks, I just pushed those patches. It took me some time because I'm
not sure how it could have worked the other way, or why. I enabled the
QA tests and they were passing without the patch. But this definitely
seems like the correct thing to do.

Tom