Fixes for issue #4

#4
Zanduino · Aug 10, 2017 · cb00d11 · cb00d11
1 parent 6673a46
commit cb00d11
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/INA226.cpp b/INA226.cpp
@@ -19,7 +19,7 @@ INA226_Class::~INA226_Class() {}                                              //
 /*******************************************************************************************************************
 ** Method begin() does all of the initialization work                                                             **
 *******************************************************************************************************************/
-void INA226_Class::begin(const uint8_t maxBusAmps, const uint32_t microOhmR) { // Class initializer                //
+void INA226_Class::begin(const uint8_t maxBusAmps, const uint32_t microOhmR){ // Class initializer                //
   Wire.begin();                                                               // Start the I2C wire subsystem     //
   for(_DeviceAddress = 1;_DeviceAddress<127;_DeviceAddress++) {               // Loop for each possible address   //
     Wire.beginTransmission(_DeviceAddress);                                   // See if something is at address   //

diff --git a/INA226.h b/INA226.h
@@ -33,7 +33,8 @@
 **                                                                                                                **
 ** Vers.  Date       Developer           Comments                                                                 **
 ** ====== ========== =================== ======================================================================== **
-** 1.0.3  2017-08-09 [email protected] Work on GitHub Bug #4 with overflow values where not expected            **
+** 1.0.3  2017-08-09 [email protected] Fix https://github.com/SV-Zanshin/INA226/issues/4. Overflows in compu-   **
+**                                       tations of begin() and getShuntMicroVolts() functions.                   **
 ** 1.0.2  2017-07-30 [email protected] Optional parameter default values only in function prototypes            **
 ** 1.0.1  2017-05-26 [email protected] Changed _CurrentLSB from uint16_t to uint32_t due to overflow bug        **
 ** 1.0.0  2017-01-10 [email protected] Fixed library file name, added constants for setMode() call              **
@@ -72,7 +73,6 @@
   const uint8_t  INA_MODE_CONTINUOUS_SHUNT    =   B101;                       // Continuous shunt, no bus         //
   const uint8_t  INA_MODE_CONTINUOUS_BUS      =   B110;                       // Continuous bus, no shunt         //
   const uint8_t  INA_MODE_CONTINUOUS_BOTH     =   B111;                       // Both continuous, default value   //
-
   /*****************************************************************************************************************
   ** Declare class header                                                                                         **
   *****************************************************************************************************************/