[lldb-dev] LLDB Evolution

Chris Lattner via lldb-dev lldb-dev at lists.llvm.org
Sun Aug 28 09:10:14 PDT 2016

Can you just grep for “^                                “ or something?  That seems like a straight-forward way to find lines that have a ton of leading indentation.


> On Aug 27, 2016, at 9:28 AM, Zachary Turner <zturner at google.com> wrote:
> 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 <mailto:clattner at apple.com>> wrote:
>> On Aug 26, 2016, at 6:12 PM, Zachary Turner via lldb-dev <lldb-dev at lists.llvm.org <mailto: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 <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/20160828/9a69d8ff/attachment.html>

More information about the lldb-dev mailing list