<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 18, 2013 at 2:36 PM, Rafaël Carré <span dir="ltr"><<a href="mailto:funman@videolan.org" target="_blank">funman@videolan.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 18/09/2013 20:52, Steve Borho a écrit :<br>
<div><div class="h5">> # HG changeset patch<br>
> # User Steve Borho <<a href="mailto:steve@borho.org">steve@borho.org</a>><br>
> # Date 1379525671 18000<br>
> #      Wed Sep 18 12:34:31 2013 -0500<br>
> # Node ID 165ef70557b68f44311bf689c825a4d2302de25a<br>
> # Parent  0b1216d453cfb3660fb62d4dfedb9613158974ed<br>
> TEncTop: remove debugging exit() call<br>
><br>
> diff -r 0b1216d453cf -r 165ef70557b6 source/Lib/TLibEncoder/TEncTop.cpp<br>
> --- a/source/Lib/TLibEncoder/TEncTop.cpp      Wed Sep 18 12:23:52 2013 -0500<br>
> +++ b/source/Lib/TLibEncoder/TEncTop.cpp      Wed Sep 18 12:34:31 2013 -0500<br>
> @@ -87,10 +87,8 @@<br>
>  {<br>
>      if (!primitives.sad[0])<br>
>      {<br>
> -        printf("Primitives must be initialized before encoder is created\n");<br>
> -        // we call exit() here because this should be an impossible condition when<br>
> -        // using our public API, and indicates a serious bug.<br>
> -        exit(1);<br>
> +        // this should be an impossible condition when using our public API, and indicates a serious bug.<br>
> +        x265_log(&param, X265_LOG_ERROR, "Primitives must be initialized before encoder is created\n");<br>
<br>
</div></div>What about:<br>
assert(primitives.sad[0] && "Primitives must be initialized before<br>
encoder is created");<br>
<br>
?<br>
<br>
It will print the message if the assert is triggered (and crash, which<br>
indicates a "serious bug" :)<br>
<div class="im"><br></div></blockquote><div><br></div><div>I want it to complain loudly in release mode as well.  An assert would be good though, will resend.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">
>      }<br>
><br>
>      m_frameEncoder = new FrameEncoder[param.frameNumThreads];<br>
</div>_______________________________________________<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" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Borho
</div></div>