Update HX711_ADC.cpp

[MalcolmBoura]

Efficiency and code size improvement.
The first term in most of the if statements was redundant anyway due to the preceding if but the daisy chain of if can be dispensed with completely.
Tested out of context with
#include <stdio.h>

void calculate(int samples)
{
int divBit;

printf("%d ", samples);
samples >>= 1;
for(divBit = 0; samples != 0; samples >>= 1, divBit++);
printf("%d %d\n", 1 << divBit, divBit);

}

int main()
{
calculate(3);
calculate(7);
calculate(8);
}

[olkal]

I had to look over your code a number of times to understand how it worked, but in the end I got it. Very neat.
Thank you, much appreciated!

[MalcolmBoura]

Hi Olav, Glad you like it. I realised that my change was neither maximum neatness nor maximum clarity! For maximum neatness for(samples >>= 1, divBit = 0; samples; samples >>= 1, divBit++); or for slightly more clarity divBit = 0; for(samples >>= 1; samples != 0; samples >>= 1) { divBit++; } All three versions should produce identical assembler, apart from some inconsequential changes in the order of a couple of things. No, I am not suggesting that it should be changed! Malcolm