<div dir="ltr">Hi Chen,<div>Thanks for spending your valuable time reviewing the patch and sharing the comments. We have incorporated the suggested changes and adding a few more to it. We will send you the updated patch soon :)</div><div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><font color="#0c343d" face="verdana, sans-serif">Thanks & Regards</font><div><font color="#0c343d" face="verdana, sans-serif"><b>Niranjan Kumar B</b></font></div><div><font size="1" color="#0c343d" face="verdana, sans-serif">Video Codec Engineer </font></div><div><font size="1" color="#0c343d" face="verdana, sans-serif">Media & AI Analytics</font></div><div><font face="trebuchet ms, sans-serif" color="#0c343d">+91 958 511 1449</font></div><div><a href="https://multicorewareinc.com/" style="color:rgb(17,85,204)" target="_blank"><img src="https://docs.google.com/uc?export=download&id=1kc3RJu9M8bnIf6Xa5rUw2d-eEVUsPBE5&revid=0B7tw9XJBmynaemR1VUpQUi9DVytRVW5SVkRwVTFjb1hBMUcwPQ"></a></div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Feb 27, 2020 at 2:35 PM chen <<a href="mailto:chenm003@163.com">chenm003@163.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"><div style="line-height:1.7;color:rgb(0,0,0);font-size:14px;font-family:Arial"><div><br></div><p>At 2020-02-27 16:59:18, "Niranjan Bala" <<a href="mailto:niranjan@multicorewareinc.com" target="_blank">niranjan@multicorewareinc.com</a>> wrote:</p><blockquote id="gmail-m_3412158403502013286isReplyContent" style="padding-left:1ex;margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204)"><div dir="ltr">+double computeBrightnessIntensity(pixel *inPlane, int width, int height, intptr_t stride)<br>+{<br>+    pixel* rowStart = inPlane;</div><div dir="ltr">restrict with const prefix may better.</div><div dir="ltr"><br>+    double count = 0;</div><div dir="ltr">why declare as Double?</div><div dir="ltr"><br>+<br>+    for (int i = 0; i < height; i++)<br>+    {<br>+        for (int j = 0; j < width; j++)<br>+        {<br>+            if (rowStart[j] > BRIGHTNESS_THRESHOLD)<br>+                count++;<br>+        }<br>+        rowStart += stride;<br>+    }<br>+<br>+    /* Returns the brightness percentage of the input plane */<br>+    return (count / (width * height)) * 100;<br>+}<br>+<br>+double computeEdgeIntensity(pixel *inPlane, int width, int height, intptr_t stride)<br>+{<br>+    pixel* rowStart = inPlane;<br>+    double count = 0;<br>+<br>+    for (int i = 0; i < height; i++)<br>+    {<br>+        for (int j = 0; j < width; j++)<br>+        {<br>+            if (rowStart[j] > 0)<br>+                count++;<br>+        }<br>+        rowStart += stride;<br>+    }<br>+<br>+    /* Returns the edge percentage of the input plane */<br>+    return (count / (width * height)) * 100;</div><div dir="ltr">100 is integer, multiplication with Double.</div><div dir="ltr"><br><br><div><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div></div></div></div></div></div></div></div></div></div>
</blockquote></div>_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org" target="_blank">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div>