gyp: remove semicolons (Python != JavaScript)

PR-URL: #29228
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>

tools/gyp/pylib/gyp/common_test.py

@@ -59,10 +59,10 @@ def test_platform_default(self):
     self.assertFlavor('freebsd', 'freebsd9' , {})
     self.assertFlavor('freebsd', 'freebsd10', {})
     self.assertFlavor('openbsd', 'openbsd5' , {})
-    self.assertFlavor('solaris', 'sunos5'   , {});
-    self.assertFlavor('solaris', 'sunos'    , {});
-    self.assertFlavor('linux'  , 'linux2'   , {});
-    self.assertFlavor('linux'  , 'linux3'   , {});
+    self.assertFlavor('solaris', 'sunos5'   , {})
+    self.assertFlavor('solaris', 'sunos'    , {})
+    self.assertFlavor('linux'  , 'linux2'   , {})
+    self.assertFlavor('linux'  , 'linux3'   , {})
 
   def test_param(self):
     self.assertFlavor('foobar', 'linux2' , {'flavor': 'foobar'})

tools/gyp/pylib/gyp/generator/analyzer.py

@@ -615,7 +615,7 @@ def _supplied_target_names(self):
 
   def _supplied_target_names_no_all(self):
     """Returns the supplied test targets without 'all'."""
-    result = self._supplied_target_names();
+    result = self._supplied_target_names()
     result.discard('all')
     return result
 
@@ -668,7 +668,7 @@ def find_matching_test_target_names(self):
 
   def find_matching_compile_target_names(self):
     """Returns the set of output compile targets."""
-    assert self.is_build_impacted();
+    assert self.is_build_impacted()
     # Compile targets are found by searching up from changed targets.
     # Reset the visited status for _GetBuildTargets.
     for target in self._name_to_target.itervalues():

tools/gyp/pylib/gyp/generator/cmake.py

@@ -686,7 +686,7 @@ def WriteTarget(namer, qualified_target, target_dicts, build_dir, config_to_use,
   for src in srcs:
     _, ext = os.path.splitext(src)
     src_type = COMPILABLE_EXTENSIONS.get(ext, None)
-    src_norm_path = NormjoinPath(path_from_cmakelists_to_gyp, src);
+    src_norm_path = NormjoinPath(path_from_cmakelists_to_gyp, src)
 
     if src_type == 's':
       s_sources.append(src_norm_path)

tools/gyp/pylib/gyp/generator/xcode.py

@@ -248,7 +248,7 @@ def Finalize1(self, xcode_targets, serialize_all_tests):
         targets_for_all.append(xcode_target)
 
       if target_name.lower() == 'all':
-        has_custom_all = True;
+        has_custom_all = True
 
       # If this target has a 'run_as' attribute, add its target to the
       # targets, and add it to the test targets.
@@ -639,7 +639,7 @@ def GenerateOutput(target_list, target_dicts, data, params):
     pbxp = xcp.project
 
     # Set project-level attributes from multiple options
-    project_attributes = {};
+    project_attributes = {}
     if parallel_builds:
       project_attributes['BuildIndependentTargetsInParallel'] = 'YES'
     if upgrade_check_project_version:
@@ -788,7 +788,7 @@ def GenerateOutput(target_list, target_dicts, data, params):
     # logic all happens in ninja.  Don't bother creating the extra targets in
     # that case.
     if type != 'none' and (spec_actions or spec_rules) and not ninja_wrapper:
-      support_xccl = CreateXCConfigurationList(configuration_names);
+      support_xccl = CreateXCConfigurationList(configuration_names)
       support_target_suffix = generator_flags.get(
           'support_target_suffix', ' Support')
       support_target_properties = {
@@ -1183,7 +1183,7 @@ def GenerateOutput(target_list, target_dicts, data, params):
         dest = '$(SRCROOT)/' + dest
 
       code_sign = int(copy_group.get('xcode_code_sign', 0))
-      settings = (None, '{ATTRIBUTES = (CodeSignOnCopy, ); }')[code_sign];
+      settings = (None, '{ATTRIBUTES = (CodeSignOnCopy, ); }')[code_sign]
 
       # Coalesce multiple "copies" sections in the same target with the same
       # "destination" property into the same PBXCopyFilesBuildPhase, otherwise

tools/gyp/pylib/gyp/mac_tool.py

@@ -285,7 +285,7 @@ def ExecFilterLibtool(self, *cmd_list):
   def ExecPackageIosFramework(self, framework):
     # Find the name of the binary based on the part before the ".framework".
     binary = os.path.basename(framework).split('.')[0]
-    module_path = os.path.join(framework, 'Modules');
+    module_path = os.path.join(framework, 'Modules')
     if not os.path.exists(module_path):
       os.mkdir(module_path)
     module_template = 'framework module %s {\n' \
@@ -346,7 +346,7 @@ def ExecCompileIosFrameworkHeaderMap(self, out, framework, *all_headers):
     WriteHmap(out, filelist)
 
   def ExecCopyIosFrameworkHeaders(self, framework, *copy_headers):
-    header_path = os.path.join(framework, 'Headers');
+    header_path = os.path.join(framework, 'Headers')
     if not os.path.exists(header_path):
       os.makedirs(header_path)
     for header in copy_headers:

tools/gyp/pylib/gyp/xcode_emulation.py

@@ -1076,7 +1076,7 @@ def _GetIOSPostbuilds(self, configname, output_binary):
     # Xcode expects XCTests to be copied into the TEST_HOST dir.
     if self._IsXCTest():
       source = os.path.join("${BUILT_PRODUCTS_DIR}", product_name)
-      test_host = os.path.dirname(settings.get('TEST_HOST'));
+      test_host = os.path.dirname(settings.get('TEST_HOST'))
       xctest_destination = os.path.join(test_host, 'PlugIns', product_name)
       postbuilds.extend(['ditto %s %s' % (source, xctest_destination)])
 
@@ -1093,7 +1093,7 @@ def _GetIOSPostbuilds(self, configname, output_binary):
 
     if self._IsXCTest():
       # For device xctests, Xcode copies two extra frameworks into $TEST_HOST.
-      test_host = os.path.dirname(settings.get('TEST_HOST'));
+      test_host = os.path.dirname(settings.get('TEST_HOST'))
       frameworks_dir = os.path.join(test_host, 'Frameworks')
       platform_root = self._XcodePlatformPath(configname)
       frameworks = \

tools/gyp/pylib/gyp/xcodeproj_file.py

@@ -1994,7 +1994,7 @@ def SetDestination(self, path):
 
     path_tree_match = self.path_tree_re.search(path)
     if path_tree_match:
-      path_tree = path_tree_match.group(1);
+      path_tree = path_tree_match.group(1)
       if path_tree in self.path_tree_first_to_subfolder:
         subfolder = self.path_tree_first_to_subfolder[path_tree]
         relative_path = path_tree_match.group(3)
@@ -2047,7 +2047,7 @@ def SetDestination(self, path):
             # subfolder = 16 from above
             # The second element of the path is an unrecognized variable.
             # Include it and any remaining elements in relative_path.
-            relative_path = path_tree_match.group(3);
+            relative_path = path_tree_match.group(3)
 
       else:
         # The path starts with an unrecognized Xcode variable