[libcamera-devel,v3,1/2] libcamera: Add operator<< for classes in geometry
diff mbox series

Message ID 20220419124220.1877877-1-hanlinchen@chromium.org
State Accepted
Headers show
Series
  • [libcamera-devel,v3,1/2] libcamera: Add operator<< for classes in geometry
Related show

Commit Message

Hanlin Chen April 19, 2022, 12:42 p.m. UTC 
Add operator<< for geometry classes for easier logging.

Signed-off-by: Han-Lin Chen <hanlinchen@chromium.org>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 include/libcamera/geometry.h |  8 +++++
 src/libcamera/geometry.cpp   | 65 +++++++++++++++++++++++++++++++-----
 2 files changed, 65 insertions(+), 8 deletions(-)

Comments

Laurent Pinchart April 19, 2022, 1:07 p.m. UTC | #1 
Hi Han-Lin,

Thank you for the patch.

On Tue, Apr 19, 2022 at 08:42:19PM +0800, Han-Lin Chen wrote:
> Add operator<< for geometry classes for easier logging.
> 
> Signed-off-by: Han-Lin Chen <hanlinchen@chromium.org>
> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
>  include/libcamera/geometry.h |  8 +++++
>  src/libcamera/geometry.cpp   | 65 +++++++++++++++++++++++++++++++-----
>  2 files changed, 65 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..da08045e 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,18 @@ bool operator==(const Point &lhs, const Point &rhs)
>   * \return True if the two points are not equal, false otherwise
>   */
>  
> +/**
> + * \brief Insert a text representation of a Point into an output stream
> + * \param[in] out The output stream
> + * \param[in] p The point
> + * \return The output stream \a out
> + */
> +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 +135,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 +442,18 @@ bool operator<(const Size &lhs, const Size &rhs)
>   * \sa bool operator<(const Size &lhs, const Size &rhs)
>   */
>  
> +/**
> + * \brief Insert a text representation of a Size into an output stream
> + * \param[in] out The output stream
> + * \param[in] sr The size

s/sr/s/

I'll fix this when applying.

> + * \return The output stream \a out
> + */
> +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 +554,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 +574,20 @@ bool operator==(const SizeRange &lhs, const SizeRange &rhs)
>   * \return True if the two size ranges are not equal, false otherwise
>   */
>  
> +/**
> + * \brief Insert a text representation of a SizeRange into an output stream
> + * \param[in] out The output stream
> + * \param[in] sr The size range
> + * \return The output stream \a out
> + */
> +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 +662,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 +833,16 @@ bool operator==(const Rectangle &lhs, const Rectangle &rhs)
>   * \return True if the two rectangles are not equal, false otherwise
>   */
>  
> +/**
> + * \brief Insert a text representation of a Rectangle into an output stream
> + * \param[in] out The output stream
> + * \param[in] r The rectangle
> + * \return The output stream \a out
> + */
> +std::ostream &operator<<(std::ostream &out, const Rectangle &r)
> +{
> +	out << "(" << r.x << "x" << r.y << ")/" << r.width << "x" << r.height;
> +	return out;
> +}
> +
>  } /* namespace libcamera */
Nicolas Dufresne via libcamera-devel April 20, 2022, 6:11 a.m. UTC | #2 
Hi Han-Lin,

Thanks for the patch.

On Tue, Apr 19, 2022 at 08:42:19PM +0800, Han-Lin Chen via libcamera-devel wrote:
> Add operator<< for geometry classes for easier logging.
> 
> Signed-off-by: Han-Lin Chen <hanlinchen@chromium.org>
> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>

> ---
>  include/libcamera/geometry.h |  8 +++++
>  src/libcamera/geometry.cpp   | 65 +++++++++++++++++++++++++++++++-----
>  2 files changed, 65 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..da08045e 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,18 @@ bool operator==(const Point &lhs, const Point &rhs)
>   * \return True if the two points are not equal, false otherwise
>   */
>  
> +/**
> + * \brief Insert a text representation of a Point into an output stream
> + * \param[in] out The output stream
> + * \param[in] p The point
> + * \return The output stream \a out
> + */
> +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 +135,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 +442,18 @@ bool operator<(const Size &lhs, const Size &rhs)
>   * \sa bool operator<(const Size &lhs, const Size &rhs)
>   */
>  
> +/**
> + * \brief Insert a text representation of a Size into an output stream
> + * \param[in] out The output stream
> + * \param[in] sr The size
> + * \return The output stream \a out
> + */
> +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 +554,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 +574,20 @@ bool operator==(const SizeRange &lhs, const SizeRange &rhs)
>   * \return True if the two size ranges are not equal, false otherwise
>   */
>  
> +/**
> + * \brief Insert a text representation of a SizeRange into an output stream
> + * \param[in] out The output stream
> + * \param[in] sr The size range
> + * \return The output stream \a out
> + */
> +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 +662,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 +833,16 @@ bool operator==(const Rectangle &lhs, const Rectangle &rhs)
>   * \return True if the two rectangles are not equal, false otherwise
>   */
>  
> +/**
> + * \brief Insert a text representation of a Rectangle into an output stream
> + * \param[in] out The output stream
> + * \param[in] r The rectangle
> + * \return The output stream \a out
> + */
> +std::ostream &operator<<(std::ostream &out, const Rectangle &r)
> +{
> +	out << "(" << r.x << "x" << r.y << ")/" << r.width << "x" << r.height;
> +	return out;
> +}
> +
>  } /* namespace libcamera */
> -- 
> 2.36.0.rc0.470.gd361397f0d-goog
>

Patch
diff mbox series 

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..da08045e 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,18 @@  bool operator==(const Point &lhs, const Point &rhs)
  * \return True if the two points are not equal, false otherwise
  */
 
+/**
+ * \brief Insert a text representation of a Point into an output stream
+ * \param[in] out The output stream
+ * \param[in] p The point
+ * \return The output stream \a out
+ */
+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 +135,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 +442,18 @@  bool operator<(const Size &lhs, const Size &rhs)
  * \sa bool operator<(const Size &lhs, const Size &rhs)
  */
 
+/**
+ * \brief Insert a text representation of a Size into an output stream
+ * \param[in] out The output stream
+ * \param[in] sr The size
+ * \return The output stream \a out
+ */
+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 +554,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 +574,20 @@  bool operator==(const SizeRange &lhs, const SizeRange &rhs)
  * \return True if the two size ranges are not equal, false otherwise
  */
 
+/**
+ * \brief Insert a text representation of a SizeRange into an output stream
+ * \param[in] out The output stream
+ * \param[in] sr The size range
+ * \return The output stream \a out
+ */
+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 +662,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 +833,16 @@  bool operator==(const Rectangle &lhs, const Rectangle &rhs)
  * \return True if the two rectangles are not equal, false otherwise
  */
 
+/**
+ * \brief Insert a text representation of a Rectangle into an output stream
+ * \param[in] out The output stream
+ * \param[in] r The rectangle
+ * \return The output stream \a out
+ */
+std::ostream &operator<<(std::ostream &out, const Rectangle &r)
+{
+	out << "(" << r.x << "x" << r.y << ")/" << r.width << "x" << r.height;
+	return out;
+}
+
 } /* namespace libcamera */