Avoid deeply nested control flow statements.

Open

            if hasattr(dependency_xct, 'test_runner'):
              all_run_tests.append(dependency_xct.test_runner)

          # Directly depend on all the runners as they depend on the target
          # that builds them.

Avoid deeply nested control flow statements.

Open

          for concrete_output_index in xrange(0, len(concrete_outputs)):
            concrete_output = concrete_outputs[concrete_output_index]
            if concrete_output_index == 0:
              bol = ''
            else:

Avoid deeply nested control flow statements.

Open

          if not item.startswith('$(BUILT_PRODUCTS_DIR)/'):
            pbxp.AddOrGetFileInRootGroup(item)

Avoid deeply nested control flow statements.

Open

            if new_dep not in extra_link_deps:
              extra_link_deps.add(new_dep)
              link_deps.append(new_dep)

Function _ExcludeFilesFromBeingBuilt has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def _ExcludeFilesFromBeingBuilt(p, spec, excluded_sources, excluded_idl,
                                list_excluded):
  exclusions = _GetExcludedFilesFromBuild(spec, excluded_sources, excluded_idl)
  for file_name, excluded_configs in exclusions.iteritems():
    if (not list_excluded and

• Read up
  Read up

Function Write has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

  def Write(self):
    # Write the project file to a temporary location first.  Xcode watches for
    # changes to the project file and presents a UI sheet offering to reload
    # the project when it does change.  However, in some cases, especially when
    # multiple projects are open or when Xcode is busy, things don't work so

• Read up
  Read up

Function _BuildCommandLineForRuleRaw has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

def _BuildCommandLineForRuleRaw(spec, cmd, cygwin_shell, has_input_path,

Avoid deeply nested control flow statements.

Open

          if precompiled_source != '':
            precompiled_source = _FixPath(precompiled_source)
            if not extensions_excluded_from_precompile:
              # If the precompiled header is generated by a C source, we must
              # not try to use it for C++ sources, and vice versa.

Function ExpandRuleVariables has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

  def ExpandRuleVariables(self, path, root, dirname, source, ext, name):

Function ComputeOutputParts has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

  def ComputeOutputParts(self, spec):
    """Return the 'output basename' of a gyp spec, split into filename + ext.

    Android libraries must be named the same thing as their module name,
    otherwise the linker can't find them, so product_name and so on must be

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

          if precompiled_source == source:
            condition = _GetConfigurationCondition(config_name, configuration)
            detail.append(['PrecompiledHeader',
                           {'Condition': condition},
                           'Create'

Avoid deeply nested control flow statements.

Open

          if concrete_output_index == len(concrete_outputs_by_rule_source) - 1:
            eol = ''
          else:
            eol = ' \\'
          makefile.write('    %s%s\n' % (concrete_output, eol))

Function WriteActionsRulesCopies has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

  def WriteActionsRulesCopies(self, spec, extra_sources, prebuild,
                              mac_bundle_depends):
    """Write out the Actions, Rules, and Copies steps.  Return a path
    representing the outputs of these steps."""
    outputs = []

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

          if message:
            # Mark it with note: so Xcode picks it up in build output.
            makefile.write('\t@echo note: %s\n' % message)
          makefile.write('\t%s\n' % action)

Function _GenerateExternalRules has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

def _GenerateExternalRules(rules, output_dir, spec,

Function _CreateProjectObjects has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def _CreateProjectObjects(target_list, target_dicts, options, msvs_version):
  """Create a MSVSProject object for the targets found in target list.

  Arguments:
    target_list: the list of targets to generate project objects for.

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

            for run_test_target in all_run_tests:
              run_all_target.AddDependency(run_test_target)

            # Insert the test runner after the related target.
            idx = self.project._properties['targets'].index(xcode_target)

Avoid deeply nested control flow statements.

Open

          for config_name, configuration in sorted(excluded_configurations):
            condition = _GetConfigurationCondition(config_name, configuration)
            detail.append(['ExcludedFromBuild',
                           {'Condition': condition},
                           'true'])

Function GypPathToNinja has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

  def GypPathToNinja(self, path, env=None):
    """Translate a gyp path to a ninja path, optionally expanding environment
    variable references in |path| with |env|.

    See the above discourse on path conversions."""

• Read up
  Read up

Function ComputeAndroidModule has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

  def ComputeAndroidModule(self, spec):
    """Return the Android module name used for a gyp spec.

    We use the complete qualified target name to avoid collisions between
    duplicate targets in different directories. We also add a suffix to

• Read up
  Read up