[PATCH 04/10] wmgtemp: Fix -Waggressive-loop-optimizations compiler warning.

Doug Torrance <[email protected]>
Newsgroups gmane.compw.window-managers.windowmaker.devel
Message-ID <[email protected]>
In particular, the cpu_history array has length 59, so when i = 58,
cpu_history[i+1] is undefined.

From:
http://sources.debian.net/src/wmgtemp/1.1-3/debian/patches/fix_-Waggressive-loop-optimizations.patch/
---
 wmgtemp/src/wmgtemp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/wmgtemp/src/wmgtemp.c b/wmgtemp/src/wmgtemp.c
index 02d75a5..f4418a6 100644
--- a/wmgtemp/src/wmgtemp.c
+++ b/wmgtemp/src/wmgtemp.c
@@ -428,7 +428,7 @@ void update_sensor_data() {
   double sys_high = highest_temp(sys_history);
 
   /* Shift the arrays */
-  for(i = 0; i < 59; i++) {
+  for(i = 0; i < 58; i++) {
     cpu_history[i] = cpu_history[i + 1];
     sys_history[i] = sys_history[i + 1];
   }
-- 
2.7.4


-- 
To unsubscribe, send mail to [email protected].
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.