Fix the implementation of the hardware brightness control on PC-BSD. Now it works consistently.

1 files changed, 5 insertions, 3 deletions

diff --git a/libLumina/LuminaOS-FreeBSD.cpp b/libLumina/LuminaOS-FreeBSD.cpp
index 73637048..430a31fc 100644
--- a/libLumina/LuminaOS-FreeBSD.cpp
+++ b/libLumina/LuminaOS-FreeBSD.cpp
@@ -8,6 +8,7 @@
 #include "LuminaOS.h"
 #include <unistd.h>
 
+#include <QDebug>
 //can't read xbrightness settings - assume invalid until set
 static int screenbrightness = -1;
 
@@ -71,17 +72,18 @@ void LOS::setScreenBrightness(int percent){
   //ensure bounds
   if(percent<0){percent=0;}
   else if(percent>100){ percent=100; }
-  float pf = percent/100.0; //convert to a decimel
   //Run the command(s)
   bool success = false;
   // - try hardware setting first (PC-BSD only)
   if(QFile::exists("/usr/local/bin/pc-sysconfig")){
-    QString ret = LUtils::getCmdOutput("pc-sysconfig \"setscreenbrightness "+QString::number(percent)+"\"").join("");
-    success = (ret.simplified() == "[SUCCESS]");
+    QString ret = LUtils::getCmdOutput("pc-sysconfig", QStringList() <<"setscreenbrightness "+QString::number(percent)).join("");
+    success = ret.toLower().contains("success");
+    qDebug() << "Set hardware brightness:" << percent << success;
   }
   // - if hardware brightness does not work, use software brightness
   if(!success){
     QString cmd = "xbrightness  %1";
+    float pf = percent/100.0; //convert to a decimel
     cmd = cmd.arg( QString::number( int(65535*pf) ) );
     success = (0 == LUtils::runCmd(cmd) );
   }