[lldb-dev] LLDB Evolution

[Zachary Turner via lldb-dev]

It will probably be hard to find all the cases.  Unfortunately clang-tidy
doesn't have a "detect deep indentation" check, but that would be pretty
useful, so maybe I'll try to add that at some point (although I doubt I can
get to it before the big reformat).

Finding all of the egregious cases before the big reformat will present a
challenge, so I'm not sure if it's better to spend effort trying, or just
deal with it as we spot code that looks bad because of indentation level.

On Sat, Aug 27, 2016 at 9:24 AM Chris Lattner <clattner at apple.com> wrote:

> On Aug 26, 2016, at 6:12 PM, Zachary Turner via lldb-dev <
> lldb-dev at lists.llvm.org> wrote:
>
> Back to the formatting issue, there's a lot of code that's going to look
> bad after the reformat, because we have some DEEPLY indented code.  LLVM
> has adopted the early return model for this reason.  A huge amount of our
> deeply nested code could be solved by using early returns.
>
>
> FWIW, early returns are part of the LLVM Coding standard:
>
> http://llvm.org/docs/CodingStandards.html#use-early-exits-and-continue-to-simplify-code
>
> So it makes sense for LLDB to adopt this approach at some point.
>
> I don’t have an opinion about whether it happens before or after the "big
> reformat", but I guess I agree with your point that doing it would be good
> to do it for the most egregious cases before the reformat.
>
> -Chris
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20160827/9e2c418f/attachment.html>