From: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Date: Fri, 21 May 2010 02:00:41 +0000 (+0000)
Subject: change calculation to require less calls, possibly fix a bug
X-Git-Tag: submit/efl/20131021.015651~7402
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=20afd3f85e5a6e1198830e1ece44e2c894552cc8;p=platform%2Fupstream%2Fenlightenment.git

change calculation to require less calls, possibly fix a bug


SVN revision: 49080
---

diff --git a/src/modules/temperature/e_mod_udev.c b/src/modules/temperature/e_mod_udev.c
index 7854401..49ffe42 100644
--- a/src/modules/temperature/e_mod_udev.c
+++ b/src/modules/temperature/e_mod_udev.c
@@ -17,7 +17,7 @@ temperature_udev_update(void *data)
    char *syspath;
    const char *test;
    char buf[256];
-   int cpus = 0;
+   double cpus = 0;
 
    inst = data;
    temp = -999;
@@ -41,7 +41,10 @@ temperature_udev_update(void *data)
         EINA_LIST_FOREACH(inst->tempdevs, l, syspath)
           {/*FIXME: probably should make a function to count the cpus and loop this */
              if ((test = eeze_udev_syspath_get_sysattr(syspath, "temp1_input")))
-               temp += (strtod(test, NULL) / 1000); /* udev reports temp in (celcius * 1000) for some reason */
+               {
+                  temp += (strtod(test, NULL) / 1000); /* udev reports temp in (celcius * 1000) for some reason */
+                  cpus++;
+               }
              if ((test = eeze_udev_syspath_get_sysattr(syspath, "temp2_input")))
                {
                   temp += (strtod(test, NULL) / 1000); /* udev reports temp in (celcius * 1000) for some reason */
@@ -53,7 +56,7 @@ temperature_udev_update(void *data)
                   cpus++;
                }
           }
-        temp /= (double)(eina_list_count(inst->tempdevs) + cpus);
+        temp /= cpus;
      }
    if (temp != -999)
      {