repo: Cleaned up pylint/pep8 violations
repo: Cleaned up pylint/pep8 violations

I noticed when running pylint (as the SUBMITTING_PATCHES file directs)
that there were a number of violations reported. This makes it difficult
to see violations I might have introduced. This commit corrects all
pylint violations in the repo script.

First I ran this to clean up the formatting:

 autopep8 --max-line-length=80 --indent-size 2 repo

Following that the following violations remained:

% pylint --rcfile=.pylintrc repo
************* Module repo
W:220,21: Redefining name 'init_optparse' from outer scope (line 156)
(redefined-outer-name)
W:482, 2: No exception type(s) specified (bare-except)
C:704, 0: Old-style class defined. (old-style-class)

For line 220, the parameter to _GitcInitOptions was renamed so as not to
mask the init_optparse global.

For line 482, a pylint directive was added to disable the bare-execpt
violation for just that line.

For line 704, the _Options class was changed to subclass object.

Additionally, the comments at lines 107-113 were spaced out to line up
with the comment at line 112 that autopep8 moved.

This script now has a pylint score of 10.0

Change-Id: I779b66eb6b061a195d3c4372b99dec1b6d2a214f

wavecomp/git-repo,aosp-mirror/tools_repo,frogbywyplay/git-repo,dodocat/git-repo,codetutil/git-repo,TheQtCompany/git-repo,gabbayo/git-repo,couchbasedeps/git-repo,codetutil/git-repo,11NJ/git-repo,jingyu/git-repo,TheQtCompany/git-repo,aosp-mirror/tools_repo,testbetta/git-repo-pub,testbetta/repo-pub,qioixiy/git-repo,lipro-yocto/git-repo,qioixiy/git-repo,chenzilin/git-repo,11NJ/git-repo,gabbayo/git-repo,chenzilin/git-repo,testbetta/repo-pub,GerritCodeReview/git-repo,frogbywyplay/git-repo,mer-tools/git-repo,jingyu/git-repo,lipro-yocto/git-repo,lewixliu/git-repo,lewixliu/git-repo,mer-tools/git-repo,wavecomp/git-repo,chzhong/git-repo,testbetta/git-repo-pub,nuclearmistake/repo,dodocat/git-repo,nuclearmistake/repo,GerritCodeReview/git-repo,couchbasedeps/git-repo,chzhong/git-repo