<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 8, 2019 at 12:27 AM Pradeep Ramachandran <<a href="mailto:pradeep@multicorewareinc.com">pradeep@multicorewareinc.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"># HG changeset patch<br>
# User Pradeep Ramachandran <<a href="mailto:pradeep@multicorewareinc.com" target="_blank">pradeep@multicorewareinc.com</a>><br>
# Date 1552022473 -19800<br>
#      Fri Mar 08 10:51:13 2019 +0530<br>
# Node ID 636258ebc7a90e0a35466e9b605ab335b9ce2194<br>
# Parent  0eccd62725b6a24ae27d52189c4a624dffdd7a07<br>
Backed out changeset: fef63866bb60<br>
<br>
diff -r 0eccd62725b6 -r 636258ebc7a9 source/encoder/encoder.cpp<br>
--- a/source/encoder/encoder.cpp        Mon Mar 04 15:36:38 2019 +0530<br>
+++ b/source/encoder/encoder.cpp        Fri Mar 08 10:51:13 2019 +0530<br>
@@ -2459,13 +2459,10 @@<br>
<br>
     if (m_param->bEmitHDRSEI)<br>
     {<br>
-        if (m_emitCLLSEI)<br>
-        {<br>
-            SEIContentLightLevel cllsei;<br>
-            cllsei.max_content_light_level = m_param->maxCLL;<br>
-            cllsei.max_pic_average_light_level = m_param->maxFALL;<br>
-            cllsei.writeSEImessages(bs, m_sps, NAL_UNIT_PREFIX_SEI, list, m_param->bSingleSeiNal);<br>
-        }<br>
+        SEIContentLightLevel cllsei;<br>
+        cllsei.max_content_light_level = m_param->maxCLL;<br>
+        cllsei.max_pic_average_light_level = m_param->maxFALL;<br>
+        cllsei.writeSEImessages(bs, m_sps, NAL_UNIT_PREFIX_SEI, list, m_param->bSingleSeiNal);<br>
<br>
         if (m_param->masteringDisplayColorVolume)<br>
         {<br></blockquote><div><br></div><div>Why?<br></div><div><br></div><div>It would be *really* nice if this kind of information was provided in the commit message without having to ask it every time.</div><div>Like in the commit that is being removed: "Some devices render out-of-luminance pixels incorrectly otherwise."</div><div><br></div><div>So, NAK until further explanation is provided.<br></div></div>-- <br><div dir="ltr" class="gmail_signature">Vittorio</div></div></div>