fix an uninit'd variable

[chiark-utils.git] / cprogs / xacpi-simple.c

diff --git a/cprogs/xacpi-simple.c b/cprogs/xacpi-simple.c
index bf0cf7863f82f343c514beb0673e0b09d95e6217..f2ba491d93adbc41d8773eedf77b4564f90ece13 100644 (file)
--- a/cprogs/xacpi-simple.c
+++ b/cprogs/xacpi-simple.c
@@ -357,7 +357,7 @@ static double total_##f;
   ALL_ACCUMULATE_FIELDS(Q_VAR)
 
 static void acquiredata(void) {
   ALL_ACCUMULATE_FIELDS(Q_VAR)
 
 static void acquiredata(void) {

-  DIR *di;
+  DIR *di = 0;

   struct dirent *de;
   int r;
   
   struct dirent *de;
   int r;
   


@@ -427,7 +427,7 @@ ALL_PLAIN_ACCUMULATE_FIELDS(Q_ACCUMULATE_PLAIN)
 
       
   }
 
       
   }

-  closedir(di);
+  if (di) closedir(di);

 
   if (debug) {
     printf("TOTAL:\n");
 
   if (debug) {
     printf("TOTAL:\n");


@@ -467,17 +467,17 @@ static void initacquire(void) {
 /*---------- argument parsing ----------*/
 
 #define COLOURS                                        \
 /*---------- argument parsing ----------*/
 
 #define COLOURS                                        \

-  C(blue,      discharging)                    \
-  C(green,     charging)                       \
-  C(cyan,      charged)                                \
-  C(lightgrey,  notcharging)                   \
-  C(grey,      confusing)                      \
-  C(black,     normal)                         \
-  C(red,       low)                            \
-  C(dimgrey,   degraded)                       \
-  C(darkgreen, absent)                         \
-  C(yellow,    error)                          \
-  C(white,     equilibrium)                    \
+  C(blue,           discharging)               \
+  C(green,         charging)                   \
+  C(cyan,           charged)                   \
+  C(lightgrey,      notcharging)               \
+  C(grey,           confusing)                 \
+  C(black,          normal)                    \
+  C(red,            low)                       \
+  C(dimgrey,        degraded)                  \
+  C(darkgreen,      absent)                    \
+  C(yellow,         error)                     \
+  C(white,          equilibrium)               \

   GC(remain)                                   \
   GC(white)                                    \
   GC(empty)
   GC(remain)                                   \
   GC(white)                                    \
   GC(empty)


@@ -576,7 +576,7 @@ typedef struct {
   unsigned long lastfg;
 } Gcstate;
 
   unsigned long lastfg;
 } Gcstate;
 

-#define C(c,u) static unsigned long pix_##c;
+#define C(c,u) static unsigned long pix_##u;

 #define GC(g) static Gcstate gc_##g;
   COLOURS
 #undef C
 #define GC(g) static Gcstate gc_##g;
   COLOURS
 #undef C


@@ -622,23 +622,23 @@ static void show(void) {
   int i, leftmost_lit, leftmost_nondeg, beyond, first_beyond;
 
   if (!charging_mask)
   int i, leftmost_lit, leftmost_nondeg, beyond, first_beyond;
 
   if (!charging_mask)

-    return show_solid(pix_darkgreen);
+    return show_solid(pix_absent);

 
   if (charging_mask & (1u << CHGST_ERROR))
 
   if (charging_mask & (1u << CHGST_ERROR))

-    return show_solid(pix_yellow);
+    return show_solid(pix_error);

 
 

-  setbackground(pix_dimgrey);
+  setbackground(pix_degraded);

   XClearWindow(disp,win);
   
   setforeground(&gc_remain,
                !(charging_mask & CHGMASK_CHG_DIS) ?
                (~charging_mask & (1u << CHGST_CHARGED) ?
   XClearWindow(disp,win);
   
   setforeground(&gc_remain,
                !(charging_mask & CHGMASK_CHG_DIS) ?
                (~charging_mask & (1u << CHGST_CHARGED) ?

-                pix_lightgrey : pix_cyan) :
-               !(~charging_mask & CHGMASK_CHG_DIS) ? pix_grey :
+                pix_notcharging : pix_charged) :
+               !(~charging_mask & CHGMASK_CHG_DIS) ? pix_confusing :

                charging_mask & (1u<<CHGST_CHARGING)
                charging_mask & (1u<<CHGST_CHARGING)

-               ? pix_green : pix_blue);
+               ? pix_charging : pix_discharging);

                
                

-  setforeground(&gc_empty, alarmed ? pix_red : pix_black);
+  setforeground(&gc_empty, alarmed ? pix_low : pix_normal);

 
   for (i=0, first_beyond=1; i<height; i++) {
     elap= !i ? 0 :
 
   for (i=0, first_beyond=1; i<height; i++) {
     elap= !i ? 0 :


@@ -676,7 +676,7 @@ static void initgc(Gcstate *gc_r) {
   memset(&gcv,0,sizeof(gcv));
   gcv.function= GXcopy;
   gcv.line_width= 1;
   memset(&gcv,0,sizeof(gcv));
   gcv.function= GXcopy;
   gcv.line_width= 1;

-  gc_r->lastfg= gcv.foreground= pix_white;
+  gc_r->lastfg= gcv.foreground= pix_equilibrium;

   gc_r->gc= XCreateGC(disp,win, GCFunction|GCLineWidth|GCForeground, &gcv);
 }
 
   gc_r->gc= XCreateGC(disp,win, GCFunction|GCLineWidth|GCForeground, &gcv);
 }
 


@@ -724,15 +724,15 @@ static void initgraphics(int argc, char **argv) {
                           pos_x,pos_y,width,height,0,0,0);
   cmap= DefaultColormap(disp,screen);
   
                           pos_x,pos_y,width,height,0,0,0);
   cmap= DefaultColormap(disp,screen);
   

-#define C(c,u) colour(&pix_##c, #u "Color");
+#define C(c,u) colour(&pix_##u, #u "Color");

 #define GC(g) initgc(&gc_##g);
   COLOURS
 #undef C
 #undef GC
 
 #define GC(g) initgc(&gc_##g);
   COLOURS
 #undef C
 #undef GC
 

-  r= XSetWindowBackground(disp,win,pix_dimgrey);
+  r= XSetWindowBackground(disp,win,pix_degraded);

   if (!r) fail("init set background");
   if (!r) fail("init set background");

-  lastbackground= pix_dimgrey;
+  lastbackground= pix_degraded;

 
   normal_hints->flags= PWinGravity;
   normal_hints->win_gravity= gravity;
 
   normal_hints->flags= PWinGravity;
   normal_hints->win_gravity= gravity;


@@ -763,15 +763,9 @@ static void refresh(void) {
 
 static void newgeometry(void) {
   int dummy;
 
 static void newgeometry(void) {
   int dummy;

-  unsigned int udummy, gotwidth, gotheight;

   Window dummyw;
   
   Window dummyw;
   

-  XGetGeometry(disp,win, &dummyw,&dummy,&dummy, &gotwidth,&gotheight,
-              &udummy,&udummy);
-  assert(gotwidth < INT_MAX);
-  assert(gotheight < INT_MAX);
-  width = gotwidth;
-  height = gotheight;
+  XGetGeometry(disp,win, &dummyw,&dummy,&dummy, &width,&height, &dummy,&dummy);

 }
 
 static void eventloop(void) {
 }
 
 static void eventloop(void) {