Fixed PEP-8 formatting recommendations in utils module

Review Request #6667 — Created Dec. 2, 2014 and discarded

Information

RBTools
master

Reviewers

This fixes a series of warnings from the pep8 tool involving line lengths, too many blank lines, and imporper spacing.

Ran tests, and pep8.

Description From Last Updated

Col: 80 E501 line too long (83 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (98 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (82 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (104 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (103 > 79 characters)

reviewbotreviewbot

'win32com' imported but unused

reviewbotreviewbot

'windll' imported but unused

reviewbotreviewbot

'com' imported but unused

reviewbotreviewbot
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/utils/appdirs.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/utils/appdirs.py
    
    
  2. rbtools/utils/appdirs.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (83 > 79 characters)
    
  3. rbtools/utils/appdirs.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (98 > 79 characters)
    
  4. rbtools/utils/appdirs.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (82 > 79 characters)
    
  5. rbtools/utils/appdirs.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (104 > 79 characters)
    
  6. rbtools/utils/appdirs.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (103 > 79 characters)
    
  7. rbtools/utils/appdirs.py (Diff revision 1)
     
     
    Show all issues
     'win32com' imported but unused
    
  8. rbtools/utils/appdirs.py (Diff revision 1)
     
     
    Show all issues
     'windll' imported but unused
    
  9. rbtools/utils/appdirs.py (Diff revision 1)
     
     
    Show all issues
     'com' imported but unused
    
  10. 
      
chipx86
  1. We don't maintain appdirs.py. We just bundle it. We should keep it exactly as it was upstream, in order to help see differences when we go to replace the version down the road.

    In other words, this change should be discarded.

  2. 
      
justy777
Review request changed
Status:
Discarded