[Lldb-commits] [lldb] r261593 - Fix Clang-tidy modernize-use-nullptr and modernize-use-default warnings in source/Commands/CommandObjectCommands.cpp; other minor fixes.
Eugene Zelenko via lldb-commits
lldb-commits at lists.llvm.org
Mon Feb 22 16:17:33 PST 2016
On Mon, Feb 22, 2016 at 4:14 PM, Adrian McCarthy <amccarth at google.com> wrote:
> It looks like this patch reorders the #includes in a way that seems contrary
> to general practice. In particular, if foo.cpp has a header foo.h, then it
> should include foo.h before any other files. This helps ensure that foo.h
> can stand alone and won't require clients to have to figure out what else
> must be included.
>
> http://llvm.org/docs/CodingStandards.html#include-style
>
> I know LLDB doesn't always follow LLVM guidelines, but this seems pretty
> common and intentional.
I try to follow headers section comments.
Eugene.
More information about the lldb-commits
mailing list