On 11 March 2013 15:24, Peter Lieven <p...@dlhnet.de> wrote:
> + unsigned long d0,d1,d2,d3;
These commas should have spaces after them. Also, since
the variables are only used inside the scope of your
newly added while loop:
> - while (size & ~(BITS_PER_LONG-1)) {
> + while (size >= 4*BITS_PER_LONG) {
it would be better to declare them here.
> + d0 = *p;
> + d1 = *(p+1);
> + d2 = *(p+2);
> + d3 = *(p+3);
> + if (d0 || d1 || d2 || d3) {
> + break;
> + }
> + p+=4;
> + result += 4*BITS_PER_LONG;
> + size -= 4*BITS_PER_LONG;
> + }
> + while (size >= BITS_PER_LONG) {
> if ((tmp = *(p++))) {
> goto found_middle;
> }
thanks
-- PMM
