Using NVCtrlLib library instead of nvidia-settings program. Refactoring and

code cleanup. Changed version to 0.4.0

wmgeneral/wmgeneral.c

@@ -366,7 +366,7 @@ void setMaskXY(int x, int y) {
 /*******************************************************************************\
 |* openXwindow																   *|
 \*******************************************************************************/
-void openXwindow(int argc, char *argv[], char *pixmap_bytes[], char *pixmask_bits, int pixmask_width, int pixmask_height) {
+void openXwindow(int argc, char *argv[], char *pixmap_bytes[], const char *pixmask_bits, int pixmask_width, int pixmask_height) {
 
 	unsigned int	borderwidth = 1;
 	XClassHint		classHint;

wmgeneral/wmgeneral.h

@@ -45,7 +45,7 @@ Display		*display;
 void AddMouseRegion(int index, int left, int top, int right, int bottom);
 int CheckMouseRegion(int x, int y);
 
-void openXwindow(int argc, char *argv[], char **, char *, int, int);
+void openXwindow(int argc, char *argv[], char **, const char *, int, int);
 void RedrawWindow(void);
 void RedrawWindowXY(int x, int y);
 

wmtempnv/Makefile

@@ -9,14 +9,14 @@ OBJ = $(SRC:.c=.o)
 
 CFLAGS = -Wall -O3
 
-LIB = -L/usr/X11R6/lib -lXpm -lXext -lX11 -lsensors
+LIB = -I/usr/include/NVCtrl/ -L/usr/X11R6/lib -lXpm -lXext -lX11 -lsensors -lXNVCtrl
 
 INSTALL = /usr/bin/install
 
 CC = gcc
 
 all: $(OBJ)
-	$(CC) -o $(EXE) $(OBJ) $(LIB)
+	$(CC) $(CFLAGS) -o $(EXE) $(OBJ) $(LIB)
 	strip $(EXE)
 
 $(OBJ): %.o : %.c 

wmtempnv/wmtempnv.c

@@ -2,11 +2,10 @@
  * wmtempnv: a sensor monitor for WindowMaker. this little app is mainly based
  * on wmsensormon and other simple dockapps.
  *
- * version = 0.1
- * date: Fri Oct 31 20:41:14 CET 2008 @861 /Internet Time/
- * author: Roman 'gryf' Dobosz
- * requirements: configured lm_sensors and sensor program, nvidia-settings, cut,
- *               grep.
+ * version = 0.4
+ *
+ * requirements: configured lm_sensors, sensor program and
+ *               nvidia-settings package
  * licence: gpl
  */
 
@@ -19,37 +18,42 @@
 #include <string.h>
 #include "../wmgeneral/wmgeneral.h"
 #include "../wmgeneral/misc.h"
+#include "../wmgeneral/misc.h"
 #include "wmtempnv_master2.xpm"
 #include "wmtempnv_mask.xbm"
 
+#include <NVCtrl/NVCtrl.h>
+#include <NVCtrl/NVCtrlLib.h>
+
 #define MAXSTRLEN 8
 #define TEMP 40
 #define TEMP_OVER 47
 #define GPU_T 70
 #define GPU_T_OVER 85
+#define MAXFNAME 50
 
 void display_values(int, int, int);
-int get_temp(int core_number);
+int get_temp(int core_number, Display*);
 int get_offset(int temp, int cpu);
+Display *display;
 
 int main(int argc, char **argv){
     int temp1=0, temp2=0, temp3=0;
     /* offset is one of 0 (normal), 7 (alert), 14 (warning) */
     int offset1=0, offset2=0, offset3=0;
     int counter = 0;
+    display = XOpenDisplay(NULL);
 
-    openXwindow(argc, argv, wmtempnv_master2_xpm, wmtempnv_mask_bits, wmtempnv_mask_width, wmtempnv_mask_height);
+    openXwindow(argc, argv, wmtempnv_master2_xpm, wmtempnv_mask_bits, 
+            wmtempnv_mask_width, wmtempnv_mask_height);
 
     while(TRUE){
-		if(counter==0){
-			temp1 = get_temp(0);
+        temp1 = get_temp(0, display);
         offset1 = get_offset(temp1, 1);
-			temp2 = get_temp(1);
+        temp2 = get_temp(1, display);
         offset2 = get_offset(temp2, 1);
-			temp3 = get_temp(2);
+        temp3 = get_temp(2, display);
         offset3 = get_offset(temp3, 0);
-			counter = 200;
-		}
 
         // core 1
         copyXPMArea(0, 87 + offset1, 23, 7, 4, 7); // LCD: "CPU"
@@ -69,7 +73,7 @@ int main(int argc, char **argv){
         display_values(temp3, 28, offset3);
         RedrawWindow();
         counter--;
-		usleep(5000);
+        sleep(1);
     }
 }
 
@@ -91,7 +95,6 @@ int get_offset(int temp, int cpu){
     }
 }
 
-
 void display_values(int temp, int offset, int offset2){
     char text[5], num1, num2, num3, num4;
 
@@ -109,26 +112,31 @@ void display_values(int temp, int offset, int offset2){
     copyXPMArea(5 * num4, 65 + offset2, 5, 7, 51, 7 + offset);
 }
 
-int get_temp(int core_number){
+int get_temp(int core_number, Display *disp){
     // Core temperature. argument is core number. core no.2 is GPU
     FILE *file;
-	int core=0;
-	char cmd[] = "                                                                                           ";
-	if(core_number==2){
-		sprintf(cmd, "echo `nvidia-settings -q 'GPUCoreTemp'|grep Attribute|cut -d ':' -f 3|cut -d '.' -f 1`");
-	}else{
-		sprintf(cmd, "echo `sensors |grep 'Core %d'|cut -d ':' -f 2|cut -d '.' -f 1`", core_number);
-	}
-    file = popen(cmd, "r");
-    while (! feof(file)) {
-        char line[MAXSTRLEN + 1];
-        bzero(line, MAXSTRLEN + 1);
-        fgets(line, MAXSTRLEN, file);
-        if(line[0] != 0){
-			sscanf(line, "%d", &core);
-        }
-    }
-	pclose(file);
+    char filename[MAXFNAME];
+    int core_temp = 0;
+	Bool res;
 
-	return core;
+    if(core_number == 2){
+
+		res = XNVCTRLQueryTargetAttribute(disp,
+				NV_CTRL_TARGET_TYPE_GPU, 0, 0,
+				NV_CTRL_GPU_CORE_TEMPERATURE, &core_temp);
+
+		if (res == False) core_temp = 0;
+    }else{
+        snprintf(filename, MAXFNAME,
+                "/sys/bus/platform/devices/coretemp.0/temp%d_input",
+                core_number + 2);
+        if((file = fopen(filename, "r")) != NULL){
+            if(fscanf(file, "%d", &core_temp) != EOF){
+                core_temp = core_temp / 1000;
+                fclose(file);
             }
+        }
+    }
+    return core_temp;
+}
+

wmtempnv/wmtempnv_mask.xbm

@@ -1,6 +1,6 @@
 #define wmtempnv_mask_width 64
 #define wmtempnv_mask_height 64
-static unsigned char wmtempnv_mask_bits[] = {
+static const char wmtempnv_mask_bits[] = {
    0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
    0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
    0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,