This is an archive of the discontinued LLVM Phabricator instance.

Fix remaining Flake8 whitespace issues
ClosedPublic

Authored by thopre on Jan 15 2021, 9:09 AM.

Details

Summary

Fix the following Flake8 errors:

  • E211 (whitespace before opening parenthesis)
  • E241 (multiple spaces after comma)

Event Timeline

thopre created this revision.Jan 15 2021, 9:09 AM
thopre requested review of this revision.Jan 15 2021, 9:09 AM

This seems rather random compared with D94789... so the spacing rule isn't consistent for assignments?

This seems rather random compared with D94789... so the spacing rule isn't consistent for assignments?

I'm not too familiar with the rules but I fail to see the inconsistency since this change does not modify any space around assignment, only spaces before parenthesis.

tnfchris accepted this revision.Jan 15 2021, 9:34 AM

This seems rather random compared with D94789... so the spacing rule isn't consistent for assignments?

I'm not too familiar with the rules but I fail to see the inconsistency since this change does not modify any space around assignment, only spaces before parenthesis.

Oh indeed. I for some reason thought it changed the =.

LGTM.

This revision is now accepted and ready to land.Jan 15 2021, 9:34 AM
thopre closed this revision.Jan 15 2021, 9:45 AM