replace use of 'a.find(b) != -1' --> 'b in a'

5 files changed, 14 insertions, 16 deletions

diff --git a/build_files/buildbot/slave_test.py b/build_files/buildbot/slave_test.py
index f79ef7cf5c6..d5acc8fd7a6 100644
--- a/build_files/buildbot/slave_test.py
+++ b/build_files/buildbot/slave_test.py
@@ -31,7 +31,7 @@ builder = sys.argv[1]
 # we run from build/ directory
 blender_dir = '../blender'
 
-if builder.find('cmake') != -1:
+if "cmake" in builder:
     # cmake
     retcode = subprocess.call(['ctest', '.' '--output-on-failure'])
     sys.exit(retcode)
diff --git a/build_files/scons/config/win32-vc-config.py b/build_files/scons/config/win32-vc-config.py
index 12ca16d480a..f1597a66538 100644
--- a/build_files/scons/config/win32-vc-config.py
+++ b/build_files/scons/config/win32-vc-config.py
@@ -3,13 +3,12 @@ import subprocess
 CL_OUT = subprocess.Popen(["cl.exe"],stdout=subprocess.PIPE,stderr=subprocess.PIPE)
 CL_STDOUT, CL_STDERR = CL_OUT.communicate()
 
-if CL_STDERR.find("17.00.") == -1:
-    VC_VERSION = '9.0'
-    LCGDIR = '#../lib/windows'
-    
-else:
+if "17.00." in CL_STDERR:
     VC_VERSION = '11.0'
     LCGDIR = '#../lib/windows_vc11'
+else:
+    VC_VERSION = '9.0'
+    LCGDIR = '#../lib/windows'
 
 LIBDIR = '${LCGDIR}'
 
diff --git a/build_files/scons/config/win64-vc-config.py b/build_files/scons/config/win64-vc-config.py
index ab1c2a87d93..a13f417c36c 100644
--- a/build_files/scons/config/win64-vc-config.py
+++ b/build_files/scons/config/win64-vc-config.py
@@ -3,13 +3,12 @@ import subprocess
 CL_OUT = subprocess.Popen(["cl.exe"],stdout=subprocess.PIPE,stderr=subprocess.PIPE)
 CL_STDOUT, CL_STDERR = CL_OUT.communicate()
 
-if CL_STDERR.find("17.00.") == -1:
-    VC_VERSION = '9.0'
-    LCGDIR = '#../lib/win64'
-    
-else:
+if "17.00." in CL_STDERR:
     VC_VERSION = '11.0'
     LCGDIR = '#../lib/win64_vc11'
+else:
+    VC_VERSION = '9.0'
+    LCGDIR = '#../lib/win64'
 
 LIBDIR = '${LCGDIR}'
 
diff --git a/build_files/scons/tools/Blender.py b/build_files/scons/tools/Blender.py
index c6cb88faebc..24bdd32ed26 100644
--- a/build_files/scons/tools/Blender.py
+++ b/build_files/scons/tools/Blender.py
@@ -847,7 +847,7 @@ class BlenderEnvironment(SConsEnvironment):
 
         def list_substring(quickie, libname):
             for q in quickie:
-                if libname.find(q) != -1:
+                if q in libname:
                     return True
             return False
 
diff --git a/build_files/scons/tools/btools.py b/build_files/scons/tools/btools.py
index 90674113f47..53dd99a7971 100644
--- a/build_files/scons/tools/btools.py
+++ b/build_files/scons/tools/btools.py
@@ -674,10 +674,10 @@ def buildslave(target=None, source=None, env=None):
     if platform == 'linux':
         import platform
 
-        if env['BF_INSTALLDIR'].find('glibc27') != -1:
-            glibc="glibc27"
-        elif env['BF_INSTALLDIR'].find('glibc211') != -1:
-            glibc="glibc211"
+        if "glibc27" in env['BF_INSTALLDIR']:
+            glibc = "glibc27"
+        elif "glibc211" in env['BF_INSTALLDIR']:
+            glibc = "glibc211"
 
         bitness = platform.architecture()[0]
         if bitness == '64bit':