commit da080fe82d2fd0b8165a1fb818c44528a7ad95ff
author Sasha McIntosh <sashamcintosh@google.com> Tue Nov 26 12:17:27 2024 -0500
committer Sasha McIntosh <sashamcintosh@google.com> Thu Jan 30 13:30:17 2025 -0500
tree 1fc1b4a3c4366829f554b1f3166ff0a5e49241b1
parent 5b5835675bf202da509bef9440f09237cdf0ee0a

drm_hwcomposer: Add method GetOptionalConnectorProperty

Add conveniece method GetOptionalConnectorProperty to reduce boilerplate
/*true=*/ notation.

Change-Id: I888be527dfbecc5c7ccfdd7661e3aeb4a9af04c7
Signed-off-by: Sasha McIntosh <sashamcintosh@google.com>

drm/DrmConnector.cpp

diff --git a/drm/DrmConnector.cpp b/drm/DrmConnector.cpp
index eeec3b1..6dbe3c5 100644
--- a/drm/DrmConnector.cpp
+++ b/drm/DrmConnector.cpp
@@ -99,8 +99,7 @@
     return false;
   }
 
-  if (GetConnectorProperty("Colorspace", &colorspace_property_,
-                           /*is_optional=*/true)) {
+  if (GetOptionalConnectorProperty("Colorspace", &colorspace_property_)) {
     colorspace_property_.AddEnumToMap("Default", Colorspace::kDefault,
                                       colorspace_enum_map_);
     colorspace_property_.AddEnumToMap("SMPTE_170M_YCC", Colorspace::kSmpte170MYcc,
@@ -135,11 +134,9 @@
                                       colorspace_enum_map_);
   }
 
-  GetConnectorProperty("content type", &content_type_property_,
-                       /*is_optional=*/true);
+  GetOptionalConnectorProperty("content type", &content_type_property_);
 
-  if (GetConnectorProperty("panel orientation", &panel_orientation_,
-                           /*is_optional=*/true)) {
+  if (GetOptionalConnectorProperty("panel orientation", &panel_orientation_)) {
     panel_orientation_
         .AddEnumToMapReverse("Normal",
                              PanelOrientation::kModePanelOrientationNormal,
@@ -162,9 +159,7 @@
 }
 
 int DrmConnector::UpdateEdidProperty() {
-  return GetConnectorProperty("EDID", &edid_property_, /*is_optional=*/true)
-             ? 0
-             : -EINVAL;
+  return GetOptionalConnectorProperty("EDID", &edid_property_) ? 0 : -EINVAL;
 }
 
 auto DrmConnector::GetEdidBlob() -> DrmModePropertyBlobUnique {