[libcamera-devel] [PATCH v3 2/2] libcamera: correct the string representation of Rectangle

[paul.elder at ideasonboard.com]

Hi Han-Lin,

On Tue, Apr 19, 2022 at 08:42:20PM +0800, Han-Lin Chen via libcamera-devel wrote:
> Change the string representation of class Rectangle from
> "(top x left)/width x height" to "(top, left)/width x height".
> 
> Signed-off-by: Han-Lin Chen <hanlinchen at chromium.org>

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

> ---
>  src/libcamera/geometry.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/libcamera/geometry.cpp b/src/libcamera/geometry.cpp
> index da08045e..e00333cf 100644
> --- a/src/libcamera/geometry.cpp
> +++ b/src/libcamera/geometry.cpp
> @@ -841,7 +841,7 @@ bool operator==(const Rectangle &lhs, const Rectangle &rhs)
>   */
>  std::ostream &operator<<(std::ostream &out, const Rectangle &r)
>  {
> -	out << "(" << r.x << "x" << r.y << ")/" << r.width << "x" << r.height;
> +	out << "(" << r.x << ", " << r.y << ")/" << r.width << "x" << r.height;
>  	return out;
>  }
>  
> -- 
> 2.36.0.rc0.470.gd361397f0d-goog
>