diff options
| author | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-25 12:23:11 +0900 |
|---|---|---|
| committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-30 10:28:20 +0900 |
| commit | 1d947b30342163b723c96db563967323535fef45 (patch) | |
| tree | b6b02d02df01792b356b9e50ceeaadcaf3a2e8c5 /manifest_xml.py | |
| parent | 2d113f35460051823ea54d61c5c939565518f969 (diff) | |
| download | git-repo-1d947b30342163b723c96db563967323535fef45.tar.gz | |
Even more coding style cleanup
Fixing some more pylint warnings:
W1401: Anomalous backslash in string
W0623: Redefining name 'name' from outer scope
W0702: No exception type(s) specified
E0102: name: function already defined line n
Change-Id: I5afcdb4771ce210390a79981937806e30900a93c
Diffstat (limited to 'manifest_xml.py')
| -rw-r--r-- | manifest_xml.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/manifest_xml.py b/manifest_xml.py index dd163bed..cdee87a6 100644 --- a/manifest_xml.py +++ b/manifest_xml.py | |||
| @@ -584,7 +584,7 @@ class XmlManifest(object): | |||
| 584 | groups = '' | 584 | groups = '' |
| 585 | if node.hasAttribute('groups'): | 585 | if node.hasAttribute('groups'): |
| 586 | groups = node.getAttribute('groups') | 586 | groups = node.getAttribute('groups') |
| 587 | groups = [x for x in re.split('[,\s]+', groups) if x] | 587 | groups = [x for x in re.split(r'[,\s]+', groups) if x] |
| 588 | 588 | ||
| 589 | default_groups = ['all', 'name:%s' % name, 'path:%s' % path] | 589 | default_groups = ['all', 'name:%s' % name, 'path:%s' % path] |
| 590 | groups.extend(set(default_groups).difference(groups)) | 590 | groups.extend(set(default_groups).difference(groups)) |