<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 12, 2013 at 5:16 AM, Derek Buitenhuis <span dir="ltr"><<a href="mailto:derek.buitenhuis@gmail.com" target="_blank">derek.buitenhuis@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 9/12/2013 8:54 AM, Gopu Govindaswamy wrote:<br>
> # HG changeset patch<br>
> # User Gopu Govindaswamy <<a href="mailto:gopu@multicorewareinc.com">gopu@multicorewareinc.com</a>><br>
> # Date 1378972464 -19800<br>
> # Node ID 7be0ceae34e059849737050c52a326becd8bbf1a<br>
> # Parent  0482a5c72c21dc14d6be5f4b9c4ffe2ae574ebc3<br>
> List: Basic List Operations To Replace std::list<br>
<br>
</div>I still do not understand the benefit of rewriting a subset of std::list<br>
in C++. What's the end goal? It is not clear.<br></blockquote><div><br></div><div>The short term issue is that STL makes integration into applications that were compiled by a different compiler pretty fragile, and just having the same class not in namespace std makes those problems go away.</div>
<div><br></div><div>The medium term issue is that the encoder still has logic that inserts NALs between other NALs in the access unit, and so we can't simply concatenate NALs as we generate them, we still need a list structure.  And since we also need a list structure for the picture lists, this is the one place in the encoder (outside of vector primitives) where templating would be somewhat sane.</div>
<div><br></div><div>Long term.. I haven't looked at how this is handled in x264.  Advice welcome.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
> +#ifndef __LIST__<br>
> +#define __LIST__<br>
<br>
Header guards should always be prefixed/namespaced. __LIST__<br>
is far too generic and could very easily conflict with<br>
system headers.<br></blockquote><div><br></div><div>agreed, __LIST__ is to generic</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> +#endif // ifndef __COMMON__<br>
<br>
Wrong comment. Copypasta code is Bad™.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Derek<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Borho
</div></div>