convertTo2sComplement() needs to be removed

[torriem]

I'm a baffled as to why there's a member function called, convertTo2sComplement(). Integers of any size in C are by definition two's complement already. There's nothing to convert.

Furthermore, the result of convertTo2sComplement() for any negative value is always 2, which is clearly incorrect.

The solution is to remove the function entirely. It's not needed and no other MMA8252 library I've looked at has one. They all just write the uint8_t offset values directly to the appropriate I2C registers.

[akupila]

The code is 6 years old so it's hard to say why anything is the way it is. Looks like this is a typo that never surfaced as it only affects setOffsets. The typo is the double &&, which should be &:

Arduino-MMA8452/MMA8452.cpp

Line 517 in 2041b42

value = (0xFF && ~value) + 1;

I believe you're correct in the method being unnecessary but i'm inclined to not touch this code too much as i have no way of testing it. If you put together a PR, i'll trust you to verify it works and accept it.