<div dir="ltr"><div dir="ltr">Hi Jacopo,<div><br></div><div>Thanks for your review!</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 24 Mar 2023 at 09:13, Jacopo Mondi <<a href="mailto:jacopo.mondi@ideasonboard.com">jacopo.mondi@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Naush<br>
<br>
On Wed, Mar 22, 2023 at 01:06:09PM +0000, Naushir Patuck via libcamera-devel wrote:<br>
> Update the overloaded RegionStats::get() and RegionStats::getFloating()<br>
> member functions to return a Region struct for consistency.<br>
><br>
> Signed-off-by: Naushir Patuck <<a href="mailto:naush@raspberrypi.com" target="_blank">naush@raspberrypi.com</a>><br>
> ---<br>
>  src/ipa/raspberrypi/controller/region_stats.h | 4 ++--<br>
>  1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/src/ipa/raspberrypi/controller/region_stats.h b/src/ipa/raspberrypi/controller/region_stats.h<br>
> index 9aaf3a58a6f7..a8860dc8dba2 100644<br>
> --- a/src/ipa/raspberrypi/controller/region_stats.h<br>
> +++ b/src/ipa/raspberrypi/controller/region_stats.h<br>
> @@ -86,12 +86,12 @@ public:<br>
>               return get_(index);<br>
>       }<br>
><br>
> -     const T &get(const libcamera::Point &pos) const<br>
> +     const Region &get(const libcamera::Point &pos) const<br>
<br>
Was this working because Region has a first member of type T ?<br></blockquote><div><br></div><div>This getter was at-this-point unused.  My usage expectation seemed to be quite<br>wrong when I originally defined this :-)<br></div><div><br></div><div>Regards,</div><div>Naush</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Anyway, looks good<br>
Reviewed-by: Jacopo Mondi <<a href="mailto:jacopo.mondi@ideasonboard.com" target="_blank">jacopo.mondi@ideasonboard.com</a>><br>
<br>
Thanks<br>
  j<br>
>       {<br>
>               return get(pos.y * size_.width + pos.x);<br>
>       }<br>
><br>
> -     const T &getFloating(unsigned int index) const<br>
> +     const Region &getFloating(unsigned int index) const<br>
>       {<br>
>               if (index >= numFloatingRegions())<br>
>                       return default_;<br>
> --<br>
> 2.34.1<br>
><br>
</blockquote></div></div>