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.mitche...@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 > >
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
