[x265] [PATCH] List: Basic List Operations To Replace std::list
Derek Buitenhuis
derek.buitenhuis at gmail.com
Thu Sep 12 12:16:52 CEST 2013
On 9/12/2013 8:54 AM, Gopu Govindaswamy wrote:
> # HG changeset patch
> # User Gopu Govindaswamy <gopu at multicorewareinc.com>
> # Date 1378972464 -19800
> # Node ID 7be0ceae34e059849737050c52a326becd8bbf1a
> # Parent 0482a5c72c21dc14d6be5f4b9c4ffe2ae574ebc3
> List: Basic List Operations To Replace std::list
I still do not understand the benefit of rewriting a subset of std::list
in C++. What's the end goal? It is not clear.
> +#ifndef __LIST__
> +#define __LIST__
Header guards should always be prefixed/namespaced. __LIST__
is far too generic and could very easily conflict with
system headers.
> +#endif // ifndef __COMMON__
Wrong comment. Copypasta code is Bad™.
- Derek
More information about the x265-devel
mailing list