AdityaSantosh / aeroquad

Automatically exported from code.google.com/p/aeroquad
0 stars 0 forks source link

In readEEPROM the calls to receiver.setTransmitterSlope and receiver.setTransmitterOffset are nonsense #44

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. receiver.setTransmitterSlope and receiver.setTransmitterOffset should have a 
float value as second parameter, but are called with the eeprom address 
instead. Same bug in the receiver.setXmitFactor(XMITFACTOR_ADR) call;

Please provide any additional information below.
This bug does not hurt because after readEEPROM() is called 
receiver.initialize() is called which reads the values from the eeprom space 
again.
Reading several eeprom values both on Receiver.h and Datastorage.h is a very 
bad idea, even when it fixes the bug by coincidence :)

Original issue reported on code.google.com by al...@arcor.de on 15 Nov 2010 at 1:46

GoogleCodeExporter commented 8 years ago
Looks like this bug was fixed between r466 and r472.

Original comment by al...@arcor.de on 15 Nov 2010 at 11:49

GoogleCodeExporter commented 8 years ago

Original comment by CaranchoEngineering@gmail.com on 26 Nov 2010 at 9:43