lineagehw: hidl: livedisplay: run clang-format
authorJan Altensen <info@stricted.net>
Fri, 19 Jul 2019 23:01:03 +0000 (01:01 +0200)
committerPaul Keith <javelinanddart@gmail.com>
Sun, 21 Jul 2019 03:24:12 +0000 (05:24 +0200)
Change-Id: I2abd96bb87321c8cfb86aa155b5423665dfb5c2e

16 files changed:
lineagehw/hidl/livedisplay/.clang-format [new file with mode: 0644]
lineagehw/hidl/livedisplay/AdaptiveBacklight.cpp
lineagehw/hidl/livedisplay/AdaptiveBacklight.h
lineagehw/hidl/livedisplay/DisplayColorCalibration.h
lineagehw/hidl/livedisplay/DisplayColorCalibrationExynos.cpp
lineagehw/hidl/livedisplay/DisplayColorCalibrationExynos.h
lineagehw/hidl/livedisplay/DisplayModes.cpp
lineagehw/hidl/livedisplay/DisplayModes.h
lineagehw/hidl/livedisplay/ReadingEnhancement.cpp
lineagehw/hidl/livedisplay/ReadingEnhancement.h
lineagehw/hidl/livedisplay/SunlightEnhancement.cpp
lineagehw/hidl/livedisplay/SunlightEnhancement.h
lineagehw/hidl/livedisplay/SunlightEnhancementExynos.cpp
lineagehw/hidl/livedisplay/SunlightEnhancementExynos.h
lineagehw/hidl/livedisplay/service.cpp
lineagehw/hidl/livedisplay/serviceExynos.cpp

diff --git a/lineagehw/hidl/livedisplay/.clang-format b/lineagehw/hidl/livedisplay/.clang-format
new file mode 100644 (file)
index 0000000..ae4a451
--- /dev/null
@@ -0,0 +1,11 @@
+BasedOnStyle: Google
+AccessModifierOffset: -2
+AllowShortFunctionsOnASingleLine: Inline
+ColumnLimit: 100
+CommentPragmas: NOLINT:.*
+DerivePointerAlignment: false
+IndentWidth: 4
+PointerAlignment: Left
+TabWidth: 4
+UseTab: Never
+PenaltyExcessCharacter: 32
index aacfa231fa35059a04dd539589de974721fd8350..986ab1254cd0239b892e2dbfdf9b4ec066ec1b1f 100644 (file)
@@ -31,7 +31,7 @@ namespace livedisplay {
 namespace V2_0 {
 namespace samsung {
 
-static constexpr const char *kBacklightPath = "/sys/class/lcd/panel/power_reduce";
+static constexpr const charkBacklightPath = "/sys/class/lcd/panel/power_reduce";
 
 bool AdaptiveBacklight::isSupported() {
     std::fstream backlight(kBacklightPath, backlight.in | backlight.out);
index 35502bafcf9d0d73a35c184349bde01ce20aea04..793f101dd65991601abbd918eea4d07c60ca6040 100644 (file)
@@ -17,9 +17,9 @@
 #ifndef VENDOR_LINEAGE_LIVEDISPLAY_V2_0_ADAPTIVEBACKLIGHT_H
 #define VENDOR_LINEAGE_LIVEDISPLAY_V2_0_ADAPTIVEBACKLIGHT_H
 
-#include <vendor/lineage/livedisplay/2.0/IAdaptiveBacklight.h>
 #include <hidl/MQDescriptor.h>
 #include <hidl/Status.h>
+#include <vendor/lineage/livedisplay/2.0/IAdaptiveBacklight.h>
 
 namespace vendor {
 namespace lineage {
@@ -44,7 +44,6 @@ class AdaptiveBacklight : public IAdaptiveBacklight {
     Return<bool> setEnabled(bool enabled) override;
 
     // Methods from ::android::hidl::base::V1_0::IBase follow.
-
 };
 
 }  // namespace samsung
index 6753a3495e15eca5b249426595b064262d5680c0..b22f988bcb28244a3f6337bef4f5c35bdc8e3a99 100644 (file)
@@ -32,7 +32,7 @@ using ::android::hardware::Void;
 #define FILE_RGB "/sys/class/graphics/fb0/rgb"
 
 class DisplayColorCalibration : public IDisplayColorCalibration {
-   public:
+  public:
     bool isSupported();
 
     // Methods from ::vendor::lineage::livedisplay::V2_0::IDisplayColorCalibration follow.
index a44ab241e2a6b6b016d445aef03b88b395ca149b..d791ed73f1f2fe4f8bacd8885b1862df657a3e40 100644 (file)
@@ -32,7 +32,7 @@ namespace livedisplay {
 namespace V2_0 {
 namespace samsung {
 
-static constexpr const char *kColorPath = "/sys/class/mdnie/mdnie/sensorRGB";
+static constexpr const charkColorPath = "/sys/class/mdnie/mdnie/sensorRGB";
 
 bool DisplayColorCalibrationExynos::isSupported() {
     std::fstream rgb(kColorPath, rgb.in | rgb.out);
index c00e5770aa9ba59637ffea2d3285e5121f132d45..81c1827318a7f3898768d90daf6dfa938d51d96f 100644 (file)
@@ -17,9 +17,9 @@
 #ifndef VENDOR_LINEAGE_LIVEDISPLAY_V2_0_DISPLAYCOLORCALIBRATIONEXYNOS_H
 #define VENDOR_LINEAGE_LIVEDISPLAY_V2_0_DISPLAYCOLORCALIBRATIONEXYNOS_H
 
-#include <vendor/lineage/livedisplay/2.0/IDisplayColorCalibration.h>
 #include <hidl/MQDescriptor.h>
 #include <hidl/Status.h>
+#include <vendor/lineage/livedisplay/2.0/IDisplayColorCalibration.h>
 
 namespace vendor {
 namespace lineage {
@@ -46,7 +46,6 @@ class DisplayColorCalibrationExynos : public IDisplayColorCalibration {
     Return<bool> setCalibration(const hidl_vec<int32_t>& rgb) override;
 
     // Methods from ::android::hidl::base::V1_0::IBase follow.
-
 };
 
 }  // namespace samsung
index 03d713ea2e16aebe7f422646af6f9730e53315aa..3a2b9021c4034c6b450d552b1b1d991c3abe413e 100644 (file)
@@ -26,18 +26,19 @@ namespace livedisplay {
 namespace V2_0 {
 namespace samsung {
 
-
 static constexpr const char* kModePath = "/sys/class/mdnie/mdnie/mode";
 static constexpr const char* kModeMaxPath = "/sys/class/mdnie/mdnie/mode_max";
 static constexpr const char* kDefaultPath = "/data/vendor/display/.displaymodedefault";
 
 const std::map<int32_t, std::string> DisplayModes::kModeMap = {
+    // clang-format off
     {0, "Dynamic"},
     {1, "Standard"},
     {2, "Natural"},
     {3, "Cinema"},
     {4, "Adaptive"},
     {5, "Reading"},
+    // clang-format on
 };
 
 DisplayModes::DisplayModes() : mDefaultModeId(0) {
@@ -76,8 +77,7 @@ Return<void> DisplayModes::getDisplayModes(getDisplayModes_cb resultCb) {
         value = kModeMap.size();
     }
     for (const auto& entry : kModeMap) {
-        if (entry.first < value)
-            modes.push_back({entry.first, entry.second});
+        if (entry.first < value) modes.push_back({entry.first, entry.second});
     }
     resultCb(modes);
     return Void();
@@ -127,7 +127,6 @@ Return<bool> DisplayModes::setDisplayMode(int32_t modeID, bool makeDefault) {
     return true;
 }
 
-
 // Methods from ::android::hidl::base::V1_0::IBase follow.
 
 }  // namespace samsung
index ef539fc0a0e79e6ccdfd3a24b360ba8d084a4dfd..149edf3de43fb3d40a94913cd0458a63a26b7be3 100644 (file)
@@ -17,9 +17,9 @@
 #ifndef VENDOR_LINEAGE_LIVEDISPLAY_V2_0_DISPLAYMODES_H
 #define VENDOR_LINEAGE_LIVEDISPLAY_V2_0_DISPLAYMODES_H
 
-#include <vendor/lineage/livedisplay/2.0/IDisplayModes.h>
 #include <hidl/MQDescriptor.h>
 #include <hidl/Status.h>
+#include <vendor/lineage/livedisplay/2.0/IDisplayModes.h>
 
 namespace vendor {
 namespace lineage {
index 4c2c31b955bd344939be405559cf9e111ecabea3..3880c43ca32f1f44434b2defbde2e2ebfa3af299 100644 (file)
@@ -31,7 +31,7 @@ namespace livedisplay {
 namespace V2_0 {
 namespace samsung {
 
-static constexpr const char *kREPath = "/sys/class/mdnie/mdnie/accessibility";
+static constexpr const charkREPath = "/sys/class/mdnie/mdnie/accessibility";
 
 // Methods from ::vendor::lineage::livedisplay::V2_0::ISunlightEnhancement follow.
 bool ReadingEnhancement::isSupported() {
@@ -54,7 +54,6 @@ Return<bool> ReadingEnhancement::setEnabled(bool enabled) {
     return WriteStringToFile(enabled ? "4" : "0", kREPath, true);
 }
 
-
 // Methods from ::android::hidl::base::V1_0::IBase follow.
 
 }  // namespace samsung
index 6fa7084d7dec868c9f02c09e00997a9a0b6616c0..5b716697cfdf3cd9a2c2feda4305da19abc41b58 100644 (file)
@@ -17,9 +17,9 @@
 #ifndef VENDOR_LINEAGE_LIVEDISPLAY_V2_0_READINGENHANCEMENT_H
 #define VENDOR_LINEAGE_LIVEDISPLAY_V2_0_READINGENHANCEMENT_H
 
-#include <vendor/lineage/livedisplay/2.0/IReadingEnhancement.h>
 #include <hidl/MQDescriptor.h>
 #include <hidl/Status.h>
+#include <vendor/lineage/livedisplay/2.0/IReadingEnhancement.h>
 
 namespace vendor {
 namespace lineage {
@@ -44,7 +44,6 @@ class ReadingEnhancement : public IReadingEnhancement {
     Return<bool> setEnabled(bool) override;
 
     // Methods from ::android::hidl::base::V1_0::IBase follow.
-
 };
 
 }  // namespace samsung
index 5c3ca51e7fc903fe511abfe09e9cae219ad42018..077672ec92777cd53b30db7c0f04d7d3e539f39a 100644 (file)
@@ -14,7 +14,6 @@
  * limitations under the License.
  */
 
-
 #include <android-base/file.h>
 #include <android-base/strings.h>
 
@@ -32,8 +31,8 @@ namespace livedisplay {
 namespace V2_0 {
 namespace samsung {
 
-static constexpr const char *kHBMPath = "/sys/class/lcd/panel/panel/auto_brightness";
-static constexpr const char *kSREPath = "/sys/class/mdnie/mdnie/outdoor";
+static constexpr const charkHBMPath = "/sys/class/lcd/panel/panel/auto_brightness";
+static constexpr const charkSREPath = "/sys/class/mdnie/mdnie/outdoor";
 
 // Methods from ::vendor::lineage::livedisplay::V2_0::ISunlightEnhancement follow.
 bool SunlightEnhancement::isSupported() {
@@ -59,7 +58,7 @@ Return<bool> SunlightEnhancement::isEnabled() {
     if (mHasHBM && ReadFileToString(kHBMPath, &tmp)) {
         statusHBM = std::stoi(Trim(tmp));
     }
-    
+
     return ((statusSRE == 1 && statusHBM == 6) || statusSRE == 1);
 }
 
index a1a627fa4ea0671de9a39de1c116d63980484111..44b34b990f2eebeec9c2fd48b4f3b31b42e3ba07 100644 (file)
@@ -17,9 +17,9 @@
 #ifndef VENDOR_LINEAGE_LIVEDISPLAY_V2_0_SUNLIGHTENHANCEMENT_H
 #define VENDOR_LINEAGE_LIVEDISPLAY_V2_0_SUNLIGHTENHANCEMENT_H
 
-#include <vendor/lineage/livedisplay/2.0/ISunlightEnhancement.h>
 #include <hidl/MQDescriptor.h>
 #include <hidl/Status.h>
+#include <vendor/lineage/livedisplay/2.0/ISunlightEnhancement.h>
 
 namespace vendor {
 namespace lineage {
index a0a330cba10124d8bc53fc53a9e326766f5a7f9a..950342ada0e8425b0963b2d8a570738852bb3fd5 100644 (file)
@@ -21,7 +21,6 @@
 
 #include "SunlightEnhancementExynos.h"
 
-
 using android::base::ReadFileToString;
 using android::base::Trim;
 using android::base::WriteStringToFile;
@@ -32,7 +31,7 @@ namespace livedisplay {
 namespace V2_0 {
 namespace samsung {
 
-static constexpr const char *kLUXPath = "/sys/class/mdnie/mdnie/lux";
+static constexpr const charkLUXPath = "/sys/class/mdnie/mdnie/lux";
 
 // Methods from ::vendor::lineage::livedisplay::V2_0::ISunlightEnhancement follow.
 bool SunlightEnhancementExynos::isSupported() {
index 5bf9798c89cbeb52957b51cd5ee7bd49e67661fc..4b348618235abb29d1f3151a3bce82c00d27334a 100644 (file)
@@ -17,9 +17,9 @@
 #ifndef VENDOR_LINEAGE_LIVEDISPLAY_V2_0_SUNLIGHTENHANCEMENTEXYNOS_H
 #define VENDOR_LINEAGE_LIVEDISPLAY_V2_0_SUNLIGHTENHANCEMENTEXYNOS_H
 
-#include <vendor/lineage/livedisplay/2.0/ISunlightEnhancement.h>
 #include <hidl/MQDescriptor.h>
 #include <hidl/Status.h>
+#include <vendor/lineage/livedisplay/2.0/ISunlightEnhancement.h>
 
 namespace vendor {
 namespace lineage {
@@ -44,7 +44,6 @@ class SunlightEnhancementExynos : public ISunlightEnhancement {
     Return<bool> setEnabled(bool enabled) override;
 
     // Methods from ::android::hidl::base::V1_0::IBase follow.
-
 };
 
 }  // namespace samsung
index b59eab1c83a2edb22cc9eff3869e2a22a54ffcd5..1e97aee0dbe458df056bfd5ecdb014675a7a4b47 100644 (file)
@@ -54,13 +54,15 @@ int main() {
 
     adaptiveBacklight = new AdaptiveBacklight();
     if (adaptiveBacklight == nullptr) {
-        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL AdaptiveBacklight Iface, exiting.";
+        LOG(ERROR)
+            << "Can not create an instance of LiveDisplay HAL AdaptiveBacklight Iface, exiting.";
         goto shutdown;
     }
 
     displayColorCalibration = new DisplayColorCalibration();
     if (displayColorCalibration == nullptr) {
-        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL DisplayColorCalibration Iface, exiting.";
+        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL DisplayColorCalibration "
+                      "Iface, exiting.";
         goto shutdown;
     }
 
@@ -72,13 +74,15 @@ int main() {
 
     readingEnhancement = new ReadingEnhancement();
     if (readingEnhancement == nullptr) {
-        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL ReadingEnhancement Iface, exiting.";
+        LOG(ERROR)
+            << "Can not create an instance of LiveDisplay HAL ReadingEnhancement Iface, exiting.";
         goto shutdown;
     }
 
     sunlightEnhancement = new SunlightEnhancement();
     if (sunlightEnhancement == nullptr) {
-        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL SunlightEnhancement Iface, exiting.";
+        LOG(ERROR)
+            << "Can not create an instance of LiveDisplay HAL SunlightEnhancement Iface, exiting.";
         goto shutdown;
     }
 
@@ -87,9 +91,8 @@ int main() {
     if (adaptiveBacklight->isSupported()) {
         status = adaptiveBacklight->registerAsService();
         if (status != OK) {
-            LOG(ERROR)
-                << "Could not register service for LiveDisplay HAL AdaptiveBacklight Iface ("
-                << status << ")";
+            LOG(ERROR) << "Could not register service for LiveDisplay HAL AdaptiveBacklight Iface ("
+                       << status << ")";
             goto shutdown;
         }
     }
@@ -107,9 +110,8 @@ int main() {
     if (displayModes->isSupported()) {
         status = displayModes->registerAsService();
         if (status != OK) {
-            LOG(ERROR)
-                << "Could not register service for LiveDisplay HAL DisplayModes Iface ("
-                << status << ")";
+            LOG(ERROR) << "Could not register service for LiveDisplay HAL DisplayModes Iface ("
+                       << status << ")";
             goto shutdown;
         }
     }
@@ -136,7 +138,7 @@ int main() {
 
     LOG(INFO) << "LiveDisplay HAL service is ready.";
     joinRpcThreadpool();
-    // Should not pass this line
+// Should not pass this line
 
 shutdown:
     // In normal operation, we don't expect the thread pool to shutdown
index e1f67b5dbc44bea9e48c14ee2b8e80c551187bbf..c7395b8d0cd6bae0d1954244fb2f0b9b708e4ffc 100644 (file)
@@ -54,13 +54,15 @@ int main() {
 
     adaptiveBacklight = new AdaptiveBacklight();
     if (adaptiveBacklight == nullptr) {
-        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL AdaptiveBacklight Iface, exiting.";
+        LOG(ERROR)
+            << "Can not create an instance of LiveDisplay HAL AdaptiveBacklight Iface, exiting.";
         goto shutdown;
     }
 
     displayColorCalibrationExynos = new DisplayColorCalibrationExynos();
     if (displayColorCalibrationExynos == nullptr) {
-        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL DisplayColorCalibration Iface, exiting.";
+        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL DisplayColorCalibration "
+                      "Iface, exiting.";
         goto shutdown;
     }
 
@@ -72,13 +74,15 @@ int main() {
 
     readingEnhancement = new ReadingEnhancement();
     if (readingEnhancement == nullptr) {
-        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL ReadingEnhancement Iface, exiting.";
+        LOG(ERROR)
+            << "Can not create an instance of LiveDisplay HAL ReadingEnhancement Iface, exiting.";
         goto shutdown;
     }
 
     sunlightEnhancementExynos = new SunlightEnhancementExynos();
     if (sunlightEnhancementExynos == nullptr) {
-        LOG(ERROR) << "Can not create an instance of LiveDisplay HAL SunlightEnhancement Iface, exiting.";
+        LOG(ERROR)
+            << "Can not create an instance of LiveDisplay HAL SunlightEnhancement Iface, exiting.";
         goto shutdown;
     }
 
@@ -87,9 +91,8 @@ int main() {
     if (adaptiveBacklight->isSupported()) {
         status = adaptiveBacklight->registerAsService();
         if (status != OK) {
-            LOG(ERROR)
-                << "Could not register service for LiveDisplay HAL AdaptiveBacklight Iface ("
-                << status << ")";
+            LOG(ERROR) << "Could not register service for LiveDisplay HAL AdaptiveBacklight Iface ("
+                       << status << ")";
             goto shutdown;
         }
     }
@@ -107,9 +110,8 @@ int main() {
     if (displayModes->isSupported()) {
         status = displayModes->registerAsService();
         if (status != OK) {
-            LOG(ERROR)
-                << "Could not register service for LiveDisplay HAL DisplayModes Iface ("
-                << status << ")";
+            LOG(ERROR) << "Could not register service for LiveDisplay HAL DisplayModes Iface ("
+                       << status << ")";
             goto shutdown;
         }
     }
@@ -136,7 +138,7 @@ int main() {
 
     LOG(INFO) << "LiveDisplay HAL service is ready.";
     joinRpcThreadpool();
-    // Should not pass this line
+// Should not pass this line
 
 shutdown:
     // In normal operation, we don't expect the thread pool to shutdown