<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 18, 2013 at 7:36 AM, SF Markus Elfring <span dir="ltr"><<a href="mailto:elfring@users.sourceforge.net" target="_blank">elfring@users.sourceforge.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hello,<br>
<br>
Would you like to replace more defines for constant values by enumerations to<br>
stress their relationships?<br>
<a href="https://bitbucket.org/multicoreware/x265/src/1c66f40da3e9/source/x265.h#cl-157" target="_blank">https://bitbucket.org/multicoreware/x265/src/1c66f40da3e9/source/x265.h#cl-157</a><br>
<br>
Examples:<br>
- X265_LOG_...<br>
- X265_CPU_LEVEL_...<br></blockquote><div><br></div><div>In general yes, but I would avoid changing X265_CPU_LEVEL_* for two reasons:</div><div><br></div><div>1 - they should go away when we drop the vector class library, to be replaced with x264's CPU capability flags</div>
<div><br></div><div>2 - they are used by the preprocessor in #if statements, and enums fail in odd ways.  I actually defined those as enums at first but had to "regress" them to preprocessor defines so the code would compile correctly</div>
<div><br></div><div>Cheers</div><div><br></div></div>-- <br>Steve Borho
</div></div>