From: Daniel Martí <mvdan@mvdan.cc>
Date: Fri, 20 Dec 2013 08:34:03 +0000 (+0100)
Subject: Replace finds with pythonic terms
X-Git-Tag: 0.1~95
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=efc831727273f355b4fc692dc29c3b167efa4d56;p=fdroidserver.git

Replace finds with pythonic terms
---

diff --git a/fdroidserver/common.py b/fdroidserver/common.py
index eaebe41f..d2f3e018 100644
--- a/fdroidserver/common.py
+++ b/fdroidserver/common.py
@@ -1361,7 +1361,7 @@ def isApkDebuggable(apkfile, config):
         print "ERROR: Failed to get apk manifest information"
         sys.exit(1)
     for line in output.splitlines():
-        if line.find('android:debuggable') != -1 and not line.endswith('0x0'):
+        if 'android:debuggable' in line and not line.endswith('0x0'):
             return True
     return False
 
diff --git a/fdroidserver/metadata.py b/fdroidserver/metadata.py
index f52fc878..2deb1a4a 100644
--- a/fdroidserver/metadata.py
+++ b/fdroidserver/metadata.py
@@ -341,11 +341,10 @@ def parse_srclib(metafile, **kw):
         if not line or line.startswith("#"):
             continue
 
-        index = line.find(':')
-        if index == -1:
+        try:
+            field, value = line.split(':',1)
+        except ValueError:
             raise MetaDataException("Invalid metadata in " + metafile.name + " at: " + line)
-        field = line[:index]
-        value = line[index+1:]
 
         if field == "Subdir":
             thisinfo[field] = value.split(',')
@@ -543,11 +542,10 @@ def parse_metadata(metafile):
             if line.startswith("#"):
                 curcomments.append(line)
                 continue
-            index = line.find(':')
-            if index == -1:
+            try:
+                field, value = line.split(':',1)
+            except ValueError:
                 raise MetaDataException("Invalid metadata in " + metafile.name + " at: " + line)
-            field = line[:index]
-            value = line[index+1:]
 
             # Translate obsolete fields...
             if field == 'Market Version':
diff --git a/fdroidserver/verify.py b/fdroidserver/verify.py
index 1c0c6b46..a9f18392 100644
--- a/fdroidserver/verify.py
+++ b/fdroidserver/verify.py
@@ -101,7 +101,7 @@ def main():
                 cwd=tmp_dir, stdout=subprocess.PIPE)
             out = p.communicate()[0]
             lines = out.splitlines()
-            if len(lines) != 1 or lines[0].find('META-INF') == -1:
+            if len(lines) != 1 or 'META-INF' not in lines[0]:
                 raise Exception("Unexpected diff output - " + out)
 
             print "...successfully verified"