From: Florian Forster <octo@leeloo.lan.home.verplant.org>
Date: Tue, 17 Mar 2009 08:22:11 +0000 (+0100)
Subject: ted plugin: Fix some minor issues with Eric's patch.
X-Git-Tag: collectd-4.7.0~81^2~1
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=c9021f3b2435f5e1688c02407fb3de427697ffb8;p=collectd.git

ted plugin: Fix some minor issues with Eric's patch.

Remove trailing whitespace, for example.
---

diff --git a/src/ted.c b/src/ted.c
index 431c7368..8dc00e5a 100644
--- a/src/ted.c
+++ b/src/ted.c
@@ -164,7 +164,7 @@ static int ted_read_value(double *ret_power, double *ret_voltage)
         /* We need to see the begin sequence first. When we receive `ESCAPE
          * PKT_BEGIN', we set `package_buffer_pos' to zero to signal that
          * the beginning of the package has been found. */
-        
+
         escape_flag = 0;
         for (i = 0; i < receive_buffer_length; i++)
         {
@@ -185,7 +185,7 @@ static int ted_read_value(double *ret_power, double *ret_voltage)
                 }
                 else if  (receive_buffer[i] == PKT_END)
                 {
-                    end_flag = 1;	
+                    end_flag = 1;
                     break;
                 }
                 else
@@ -209,17 +209,17 @@ static int ted_read_value(double *ret_power, double *ret_voltage)
     } /* while (end_flag == 0) */
 
     /* Check for errors inside the loop. */
-    if ((end_flag == 0) | (package_buffer_pos != 278))
+    if ((end_flag == 0) || (package_buffer_pos != EXPECTED_PACKAGE_LENGTH))
         return (-1);
-	   
-    /* 
+
+    /*
      * Power is at positions 247 and 248 (LSB first) in [10kW].
      * Voltage is at positions 251 and 252 (LSB first) in [.1V].
      *
-     * Power is in 1Watt steps  
+     * Power is in 10 Watt steps
      * Voltage is in volts
      */
-    *ret_power = 10 * (double) ((((int) package_buffer[248]) * 256)
+    *ret_power = 10.0 * (double) ((((int) package_buffer[248]) * 256)
             + ((int) package_buffer[247]));
     *ret_voltage = 0.1 * (double) ((((int) package_buffer[252]) * 256)
             + ((int) package_buffer[251]));
@@ -265,7 +265,7 @@ static int ted_open_device (void)
 } /* int ted_open_device */
 
 static void ted_submit (char *type, double value)
-{	
+{
     value_t values[1];
     value_list_t vl = VALUE_LIST_INIT;