paleofetch

commit 488294010da7d58a415225cf56eac31fb44f38d0
parent 5084173d502fe680c2962a642511173144cc8fde
Author: Sridaran Thoniyil <sri7thon@gmail.com>
Date:   Fri,  8 May 2020 14:55:39 -0400

Optimize battery function by using length of strings for strcat


Diffstat:
M
paleofetch.c
 | 
12
+++++++-----

1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/paleofetch.c b/paleofetch.c
@@ -242,13 +242,15 @@ static char *get_battery_percentage() {
     if((battery_percentage_file = fopen(BATTERY_DIRECTORY "/capacity", "r")) != NULL) {
         // at most 100, which is 3 characters
         fread(battery_percentage, 1, 3, battery_percentage_file);
-        remove_newline(battery_percentage);
-        strcat(battery_percentage, "% [");
+        int battery_percentage_length = remove_newline_get_length(battery_percentage);
+        char* battery_percentage_end = battery_percentage + battery_percentage_length;
+        strcat(battery_percentage_end, "% [");
+        battery_percentage_end += 3;
         if((battery_status_file = fopen(BATTERY_DIRECTORY "/status", "r")) != NULL) {
             fread(battery_status, 1, 12, battery_status_file);
-            remove_newline(battery_status);
-            strcat(battery_percentage, battery_status);
-            strcat(battery_percentage, "]");
+            int battery_status_length = remove_newline_get_length(battery_status);
+            strcat(battery_percentage_end, battery_status);
+            strcat(battery_percentage_end + battery_status_length, "]");
         }
         else {
             strcat(battery_percentage, "Unknown]");