Add buffer to header file, print new brightness on command and add install target to makefile

Makefile

@@ -7,6 +7,10 @@ OBJ=$(SRC:.c=.o)
 
 HEADER=xbrightness.h
 
+install: all
+	cp $(NAME) /usr/local/bin/
+	@make clean
+
 all: $(NAME)
 
 $(NAME): $(OBJ) $(HEADER)

xbrightness.c

@@ -13,23 +13,23 @@ void cmd(char *command, char *output) {
 }
 
 float current() {
-    char out[1028];
+    char out[BUFFER];
-    cmd("xrandr --verbose | grep Brightness: | cut -f2 -d\" \"", out);
+    cmd("xrandr --verbose | grep Brightness: | cut -f2 -d\" \"", out); //TODO: Check when 0 screen connected and two or more connected and catch output error
-    return strtof(out, NULL);
+    return strtof(out, NULL); //TODO: Return NULL when convert fail
 }
 
 void set(float brightness) {
-    char screen[1028];
+    char screen[BUFFER]; //TODO: Better buffer
-    char command[1028] = "xrandr --output ";
+    char command[BUFFER] = "xrandr --output ";
     char command2[] = " --brightness ";
     char b[3];
-    cmd("xrandr | grep \" connected\" | cut -f1 -d\" \"", screen);
+    cmd("xrandr | grep \" connected\" | cut -f1 -d\" \"", screen); //TODO: Check when 0 screen connected and two or more connected and catch output error
     screen[strlen(screen)-1] = 0;
     strcat(command, screen);
     strcat(command, command2);
     gcvt(brightness, 3, b);
     strcat(command, b);
-    cmd(command, NULL);
+    cmd(command, NULL); //TODO: Catch output error
 }
 
 int main(int argc, char **argv) {
@@ -48,10 +48,12 @@ int main(int argc, char **argv) {
         return 1;
     } else {
         if (argv[1][0] == '+' ||argv[1][0] == '-') {
-            set(current()+b);
+            b = current()+b;
+            set(b);
         } else {
             set(b);
         }
+        printf("Brightness set to %.2f\n", b);
     }
     return 0;
 }

xbrightness.h

@@ -1,6 +1,8 @@
 #ifndef XBRIGHTNESS_H
 #define XBRIGHTNESS_H
 
+#define BUFFER 1028
+
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>