PowerAdapter (Linux): don't print not connected either

To align result to macOS

Author: CarterLi

src/detection/poweradapter/poweradapter.h

@@ -13,5 +13,3 @@ typedef struct FFPowerAdapterResult
 } FFPowerAdapterResult;
 
 const char* ffDetectPowerAdapter(FFlist* results);
-
-#define FF_POWERADAPTER_NOT_CONNECTED -1

src/detection/poweradapter/poweradapter_apple.c

@@ -17,7 +17,7 @@ const char* ffDetectPowerAdapter(FFlist* results)
         ffStrbufInit(&adapter->manufacturer);
         ffStrbufInit(&adapter->modelName);
         ffStrbufInit(&adapter->serial);
-        adapter->watts = FF_POWERADAPTER_NOT_CONNECTED;
+        adapter->watts = 0;
 
         ffCfDictGetString(details, CFSTR(kIOPSNameKey), &adapter->name);
         if (ffCfDictGetString(details, CFSTR("Model"), &adapter->modelName) != NULL)

src/detection/poweradapter/poweradapter_linux.c

@@ -28,45 +28,41 @@ static void parsePowerAdapter(FFstrbuf* dir, FF_MAYBE_UNUSED const char* id, FFl
     if(ffStrbufIgnCaseEqualS(&tmpBuffer, "Device"))
         return;
 
+    ffStrbufAppendS(dir, "/online");
+    char online = '1';
+    ffReadFileData(dir->chars, sizeof(online), &online);
+    ffStrbufSubstrBefore(dir, dirLength);
+
+    if (online == '0')
+        return;
+
     //input_power_limit must exist and be not empty
     ffStrbufAppendS(dir, "/input_power_limit");
     bool available = ffReadFileBuffer(dir->chars, &tmpBuffer);
     ffStrbufSubstrBefore(dir, dirLength);
 
     if (!available)
         return;
-
     FFPowerAdapterResult* result = ffListAdd(results);
     ffStrbufInit(&result->name);
     ffStrbufInit(&result->description);
-
-    ffStrbufAppendS(dir, "/online");
-    char online = '1';
-    ffReadFileData(dir->chars, sizeof(online), &online);
-    ffStrbufSubstrBefore(dir, dirLength);
-
-    if (online == '0')
-        result->watts = FF_POWERADAPTER_NOT_CONNECTED;
-    else
-        result->watts = (int) (ffStrbufToDouble(&tmpBuffer) / 1e6 + 0.5);
-
-    //At this point, we have a battery. Try to get as much values as possible.
-
+    result->watts = (int) (ffStrbufToDouble(&tmpBuffer) / 1e6 + 0.5);
     ffStrbufInit(&result->manufacturer);
+    ffStrbufInit(&result->modelName);
+    ffStrbufInit(&result->serial);
+
     ffStrbufAppendS(dir, "/manufacturer");
     if (ffReadFileBuffer(dir->chars, &result->manufacturer))
         ffStrbufTrimRightSpace(&result->manufacturer);
     else if (ffStrEquals(id, "macsmc-ac")) // asahi
         ffStrbufSetStatic(&result->manufacturer, "Apple Inc.");
     ffStrbufSubstrBefore(dir, dirLength);
 
-    ffStrbufInit(&result->modelName);
     ffStrbufAppendS(dir, "/model_name");
     if (ffReadFileBuffer(dir->chars, &result->modelName))
         ffStrbufTrimRightSpace(&result->modelName);
     ffStrbufSubstrBefore(dir, dirLength);
 
-    ffStrbufInit(&result->serial);
     ffStrbufAppendS(dir, "/serial_number");
     if (ffReadFileBuffer(dir->chars, &result->serial))
         ffStrbufTrimRightSpace(&result->serial);
@@ -95,8 +91,5 @@ const char* ffDetectPowerAdapter(FFlist* results)
         ffStrbufSubstrBefore(&baseDir, baseDirLength);
     }
 
-    if(results->length == 0)
-        return "\"/sys/class/power_supply/\" doesn't contain any power adapter folder";
-
     return NULL;
 }

src/modules/poweradapter/poweradapter.c

@@ -33,8 +33,6 @@ void ffPrintPowerAdapter(FFPowerAdapterOptions* options)
 
                 if(result->name.length > 0)
                     puts(result->name.chars);
-                else if(result->watts == FF_POWERADAPTER_NOT_CONNECTED)
-                    puts("not connected");
                 else
                     printf("%dW\n", result->watts);
             }