[libcamera-devel] [PATCH v6 21/12] libcamera: transform: Make the transformFromOrientation() function static

Laurent Pinchart laurent.pinchart at ideasonboard.com
Mon Oct 23 00:41:59 CEST 2023


Now that the transformFromOrientation() function isn't used outside of
transform.cpp, make it static to remove it from the public API.

Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
---
 include/libcamera/transform.h | 1 -
 src/libcamera/transform.cpp   | 4 ++++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/include/libcamera/transform.h b/include/libcamera/transform.h
index 4998a6c04cb7..44cb4c6fc974 100644
--- a/include/libcamera/transform.h
+++ b/include/libcamera/transform.h
@@ -71,7 +71,6 @@ constexpr Transform operator~(Transform t)
 }
 
 Transform transformFromRotation(int angle, bool *success = nullptr);
-Transform transformFromOrientation(const Orientation &orientation);
 
 Transform operator/(const Orientation &o1, const Orientation &o2);
 Orientation operator*(const Orientation &o, const Transform &t);
diff --git a/src/libcamera/transform.cpp b/src/libcamera/transform.cpp
index cd2717c28c3a..fb2d55ac1931 100644
--- a/src/libcamera/transform.cpp
+++ b/src/libcamera/transform.cpp
@@ -301,6 +301,8 @@ Transform transformFromRotation(int angle, bool *success)
 	return Transform::Identity;
 }
 
+namespace {
+
 /**
  * \brief Return the transform representing \a orientation
  * \param[in] orientation The orientation to convert
@@ -330,6 +332,8 @@ Transform transformFromOrientation(const Orientation &orientation)
 	return Transform::Identity;
 }
 
+} /* namespace */
+
 /**
  * \brief Return the Transform that applied to \a o2 gives \a o1
  * \param o1 The Orientation to obtain
-- 
Regards,

Laurent Pinchart



More information about the libcamera-devel mailing list