[Lldb-commits] [PATCH] D14375: [swig] Start of pylint on python build scripts.

[Zachary Turner via lldb-commits]

Would it be possible for you to break this up into patches that only run 1
fixer at a time?  When I look at the diff, basically every line has a
change, so it's hard for me to verify that no actual substance changed, and
that this is all just style issues.

On Wed, Nov 4, 2015 at 11:42 PM Bruce Mitchener <bruce.mitchener at gmail.com>
wrote:

> brucem created this revision.
> brucem added reviewers: zturner, domipheus.
> brucem added a subscriber: lldb-commits.
>
> This does a broad first pass on cleaning up a lot of the noise when
> using pylint on these scripts. It mostly addresses issues of:
>
> * Mixed tabs and spaces.
> * Trailing whitespace.
> * Semicolons where they aren't needed.
> * Incorrect whitespace around () and [].
> * Superfluous parentheses.
>
> There will be subsequent patches with further changes that build
> upon these.
>
> http://reviews.llvm.org/D14375
>
> Files:
>   scripts/Python/buildSwigPython.py
>   scripts/Python/finishSwigPythonLLDB.py
>   scripts/Python/modify-python-lldb.py
>   scripts/buildSwigWrapperClasses.py
>   scripts/finishSwigWrapperClasses.py
>   scripts/utilsArgsParse.py
>   scripts/utilsDebug.py
>   scripts/utilsOsType.py
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151105/a593d381/attachment.html>