<div dir="ltr"><div dir="ltr">Hi Laurent,<div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, May 16, 2021 at 3:38 AM Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@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">The addEntry() and updateEntry() overrides that take a reference to a<br>
container don't need to modify the container. Make it const.<br>
<br>
Signed-off-by: Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank">laurent.pinchart@ideasonboard.com</a>><br></blockquote><div><br></div><div>Reviewed-by: Hirokazu Honda <<a href="mailto:hiroh@chromium.org">hiroh@chromium.org</a>></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>
 src/android/camera_metadata.h | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/src/android/camera_metadata.h b/src/android/camera_metadata.h<br>
index 56df6f1c9008..f95c34581fe4 100644<br>
--- a/src/android/camera_metadata.h<br>
+++ b/src/android/camera_metadata.h<br>
@@ -42,7 +42,7 @@ public:<br>
<br>
        template<typename S,<br>
                 typename T = typename S::value_type><br>
-       bool addEntry(uint32_t tag, S &data)<br>
+       bool addEntry(uint32_t tag, const S &data)<br>
        {<br>
                return addEntry(tag, data.data(), data.size(), sizeof(T));<br>
        }<br>
@@ -67,7 +67,7 @@ public:<br>
<br>
        template<typename S,<br>
                 typename T = typename S::value_type><br>
-       bool updateEntry(uint32_t tag, S &data)<br>
+       bool updateEntry(uint32_t tag, const S &data)<br>
        {<br>
                return updateEntry(tag, data.data(), data.size());<br>
        }<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
<br>
</blockquote></div></div>