[libcamera-devel] [PATCH v2] libcamera: Add operator<< for classes in geometry
Han-Lin Chen
hanlinchen at chromium.org
Mon Apr 18 14:19:09 CEST 2022
Add operator<< for geometry classes for easier logging.
Signed-off-by: Han-Lin Chen <hanlinchen at chromium.org>
---
include/libcamera/geometry.h | 8 +++++
src/libcamera/geometry.cpp | 57 +++++++++++++++++++++++++++++++-----
2 files changed, 57 insertions(+), 8 deletions(-)
diff --git a/include/libcamera/geometry.h b/include/libcamera/geometry.h
index 7838b679..d4a144bc 100644
--- a/include/libcamera/geometry.h
+++ b/include/libcamera/geometry.h
@@ -46,6 +46,8 @@ static inline bool operator!=(const Point &lhs, const Point &rhs)
return !(lhs == rhs);
}
+std::ostream &operator<<(std::ostream &out, const Point &p);
+
class Size
{
public:
@@ -192,6 +194,8 @@ static inline bool operator>=(const Size &lhs, const Size &rhs)
return !(lhs < rhs);
}
+std::ostream &operator<<(std::ostream &out, const Size &s);
+
class SizeRange
{
public:
@@ -232,6 +236,8 @@ static inline bool operator!=(const SizeRange &lhs, const SizeRange &rhs)
return !(lhs == rhs);
}
+std::ostream &operator<<(std::ostream &out, const SizeRange &sr);
+
class Rectangle
{
public:
@@ -291,4 +297,6 @@ static inline bool operator!=(const Rectangle &lhs, const Rectangle &rhs)
return !(lhs == rhs);
}
+std::ostream &operator<<(std::ostream &out, const Rectangle &r);
+
} /* namespace libcamera */
diff --git a/src/libcamera/geometry.cpp b/src/libcamera/geometry.cpp
index cb3c2de1..a0696c51 100644
--- a/src/libcamera/geometry.cpp
+++ b/src/libcamera/geometry.cpp
@@ -56,8 +56,7 @@ namespace libcamera {
const std::string Point::toString() const
{
std::stringstream ss;
-
- ss << "(" << x << "," << y << ")";
+ ss << *this;
return ss.str();
}
@@ -83,6 +82,16 @@ bool operator==(const Point &lhs, const Point &rhs)
* \return True if the two points are not equal, false otherwise
*/
+/**
+ * \brief Insert operation for Point with ostream
+ * \return The input std::ostream
+ */
+std::ostream &operator<<(std::ostream &out, const Point &p)
+{
+ out << "(" << p.x << ", " << p.y << ")";
+ return out;
+}
+
/**
* \struct Size
* \brief Describe a two-dimensional size
@@ -124,7 +133,10 @@ bool operator==(const Point &lhs, const Point &rhs)
*/
const std::string Size::toString() const
{
- return std::to_string(width) + "x" + std::to_string(height);
+ std::stringstream ss;
+ ss << *this;
+
+ return ss.str();
}
/**
@@ -428,6 +440,16 @@ bool operator<(const Size &lhs, const Size &rhs)
* \sa bool operator<(const Size &lhs, const Size &rhs)
*/
+/**
+ * \brief Insert operation for Size with ostream
+ * \return The input std::ostream
+ */
+std::ostream &operator<<(std::ostream &out, const Size &s)
+{
+ out << s.width << "x" << s.height;
+ return out;
+}
+
/**
* \struct SizeRange
* \brief Describe a range of sizes
@@ -528,9 +550,7 @@ bool SizeRange::contains(const Size &size) const
std::string SizeRange::toString() const
{
std::stringstream ss;
-
- ss << "(" << min.toString() << ")-(" << max.toString() << ")/(+"
- << hStep << ",+" << vStep << ")";
+ ss << *this;
return ss.str();
}
@@ -550,6 +570,18 @@ bool operator==(const SizeRange &lhs, const SizeRange &rhs)
* \return True if the two size ranges are not equal, false otherwise
*/
+/**
+ * \brief Insert operation for SizeRange with ostream
+ * \return The input std::ostream
+ */
+std::ostream &operator<<(std::ostream &out, const SizeRange &sr)
+{
+ out << "(" << sr.min << ")-(" << sr.max << ")/(+"
+ << sr.hStep << ",+" << sr.vStep << ")";
+
+ return out;
+}
+
/**
* \struct Rectangle
* \brief Describe a rectangle's position and dimensions
@@ -624,8 +656,7 @@ bool operator==(const SizeRange &lhs, const SizeRange &rhs)
const std::string Rectangle::toString() const
{
std::stringstream ss;
-
- ss << "(" << x << "x" << y << ")/" << width << "x" << height;
+ ss << *this;
return ss.str();
}
@@ -796,4 +827,14 @@ bool operator==(const Rectangle &lhs, const Rectangle &rhs)
* \return True if the two rectangles are not equal, false otherwise
*/
+/**
+ * \brief Insert operation for Rectangle with ostream
+ * \return The input std::ostream
+ */
+std::ostream &operator<<(std::ostream &out, const Rectangle &r)
+{
+ out << "(" << r.x << ", " << r.y << ")/" << r.width << "x" << r.height;
+ return out;
+}
+
} /* namespace libcamera */
--
2.36.0.rc0.470.gd361397f0d-goog
More information about the libcamera-devel
mailing list