<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 13, 2018 at 5:23 AM, Ma0 <span dir="ltr"><<a href="mailto:mateuszb@poczta.onet.pl" target="_blank">mateuszb@poczta.onet.pl</a>></span> wrote:<br><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 Ma0 <<a href="mailto:mateuszb@poczta.onet.pl">mateuszb@poczta.onet.pl</a>><br>
# Date 1515799544 -3600<br>
#      Sat Jan 13 00:25:44 2018 +0100<br>
# Node ID 6f4c949761c446334245d72204a3cd<wbr>7345b36a9f<br>
# Parent  2f3c4158cf3553030920708271bc43<wbr>cdc79932a3<br>
param2string: increase buffer size, do not store file names<br>
<br>
diff -r 2f3c4158cf35 -r 6f4c949761c4 source/common/param.cpp<br>
--- a/source/common/param.cpp   Thu Jan 04 12:37:01 2018 +0530<br>
+++ b/source/common/param.cpp   Sat Jan 13 00:25:44 2018 +0100<br>
@@ -1530,8 +1530,14 @@<br>
 char *x265_param2string(x265_param* p, int padx, int pady)<br>
 {<br>
     char *buf, *s;<br>
+    int bufSize = 4000 + p->rc.zoneCount * 64;<br></blockquote><div><br></div><div>bufSize can be replaced with macro MAXPARAMSIZE. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
-    buf = s = X265_MALLOC(char, MAXPARAMSIZE);<br>
+    if (p->numaPools)<br>
+        bufSize += strlen(p->numaPools);<br>
+    if (p-><wbr>masteringDisplayColorVolume)<br>
+        bufSize += strlen(p-><wbr>masteringDisplayColorVolume);<br></blockquote><div><br></div><div>We have other string parameters like csv file name, analysis save and load etc., why adding string length of only two parameters in buffer size?</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>
+    buf = s = X265_MALLOC(char, bufSize);<br>
     if (!buf)<br>
         return NULL;<br>
<br>
@@ -1549,7 +1555,7 @@<br>
     BOOL(p->bEnableSsim, "ssim");<br>
     s += sprintf(s, " log-level=%d", p->logLevel);<br>
     if (p->csvfn)<br>
-        s += sprintf(s, " csvfn=%s csv-log-level=%d", p->csvfn, p->csvLogLevel);<br>
+        s += sprintf(s, " csv csv-log-level=%d", p->csvLogLevel);<br>
     s += sprintf(s, " bitdepth=%d", p->internalBitDepth);<br>
     s += sprintf(s, " input-csp=%d", p->internalCsp);<br>
     s += sprintf(s, " fps=%u/%u", p->fpsNum, p->fpsDenom);<br>
@@ -1722,8 +1728,10 @@<br>
     BOOL(p->bEmitHDRSEI, "hdr");<br>
     BOOL(p->bHDROpt, "hdr-opt");<br>
     BOOL(p->bDhdr10opt, "dhdr10-opt");<br>
-    s += sprintf(s, " analysis-save=%s", p->analysisSave);<br>
-    s += sprintf(s, " analysis-load=%s", p->analysisLoad);<br>
+    if (p->analysisSave)<br>
+        s += sprintf(s, " analysis-save");<br>
+    if (p->analysisLoad)<br>
+        s += sprintf(s, " analysis-load");<br>
     s += sprintf(s, " analysis-reuse-level=%d", p->analysisReuseLevel);<br>
     s += sprintf(s, " scale-factor=%d", p->scaleFactor);<br>
     s += sprintf(s, " refine-intra=%d", p->intraRefine);<br>
diff -r 2f3c4158cf35 -r 6f4c949761c4 source/common/param.h<br>
--- a/source/common/param.h     Thu Jan 04 12:37:01 2018 +0530<br>
+++ b/source/common/param.h     Sat Jan 13 00:25:44 2018 +0100<br>
@@ -53,8 +53,6 @@<br>
 int x265_param_parse(x265_param *p, const char *name, const char *value);<br>
 #define PARAM_NS X265_NS<br>
 #endif<br>
-<br>
-#define MAXPARAMSIZE 2000<br>
 }<br>
<br>
 #endif // ifndef X265_PARAM_H<br>
______________________________<wbr>_________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/<wbr>listinfo/x265-devel</a><br>
</blockquote></div><br></div></div>