stella.stamenova added a comment. In https://reviews.llvm.org/D54009#1284839, @zturner wrote:
> In https://reviews.llvm.org/D54009#1284827, @stella.stamenova wrote: > > > Looks good. The formatting in lit.cfg.py is a bit messy (indentations, > > especially), but as long as the tests pass, this is an improvement :). > > Thanks! > > > Is there something like clang-format for Python? Happy to fix it if so. (I > don't do a lot of Python so it's hard for me to eyeball it and figure out > what's wrong, so I have to say it looks fine to me :)) There's no clang-format for Python that I know of. Since python cares about indentation, it's always a good idea to make sure that we use the same number of spaces for all of the 'if's and such as it will occasionally lead to hard to track down bugs. There are (in the final version) of lit.cfg.py a couple of places where two spaces make the indentation and everything else uses four. It's not a big deal because of where they are, but something to look out for. Repository: rLLDB LLDB https://reviews.llvm.org/D54009 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
