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

Open

def _FindDirectXInstallation():
  """Try to find an installation location for the DirectX SDK. Check for the
  standard environment variable, and if that doesn't exist, try to find
  via the registry. May return None if not found in either location."""
  # Return previously calculated value, if there is one

• Read up
  Read up

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

Open

def MergeLists(to, fro, to_file, fro_file, is_paths=False, append=True):

Avoid deeply nested control flow statements.

Open

          if build_file_dir:  # build_file_dir may be None (see above).
            os.chdir(build_file_dir)
          try:

Avoid deeply nested control flow statements.

Open

          if t == target_name:
            if targets[t].get('variables', {}).get('prune_self_dependency', 0):
              target_dict[dependency_key] = Filter(dependencies, target_name)

Avoid deeply nested control flow statements.

Open

            for item in value:
              self._properties[property].append(item.Copy())
          else:

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

Open

def LoadBuildFileIncludesIntoList(sublist, sublist_path, data, aux_data,

Avoid deeply nested control flow statements.

Open

          if (dependency_target != '*' and
              dependency_target != dependency_target_name):
            continue
          dependency_target_toolset = dependency_target_dict['toolset']

Avoid deeply nested control flow statements.

Open

          if list_actions[index] == action_value:
            # Even if the regex matches, nothing will change so continue (regex
            # searches are expensive).
            continue
          if pattern_re.search(list_item):

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

Open

def ValidateSourcesInTarget(target, target_dict, build_file):
  # TODO: Check if MSVC allows this for loadable_module targets.
  if target_dict.get('type', None) not in ('static_library', 'shared_library'):
    return
  sources = target_dict.get('sources', [])

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

          if pattern_re.search(list_item):
            # Regular expression match.
            list_actions[index] = action_value

      # The "whatever/" list is no longer needed, dump it.

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

Open

  def __init__(self, properties=None, id=None, parent=None,

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

Open

  def __init__(self, properties=None, id=None, parent=None,

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

Open

def TurnIntIntoStrInDict(the_dict):
  """Given dict the_dict, recursively converts all integers into strings.
  """
  # Use items instead of iteritems because there's no need to try to look at
  # reinserted keys and their associated values.

• Read up
  Read up

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

Open

  def AddDependency(self, other):
    # super
    XCTarget.AddDependency(self, other)

    static_library_type = 'com.apple.product-type.library.static'

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

          if p.wait() != 0 or p_stderr:
            sys.stderr.write(p_stderr)
            # Simulate check_call behavior, since check_call only exists
            # in python 2.5 and later.
            raise GypError("Call to '%s' returned exit status %d." %

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

Open

    def rule(self, name, command, description=None, depfile=None,
             generator=False, pool=None, restat=False, rspfile=None,
             rspfile_content=None, deps=None):
        self._line('rule %s' % name)
        self.variable('command', command, indent=1)

• Read up
  Read up

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

Open

  def HasRun(self, output):
    total = self.succeeded + len(self.failed)
    if (total > 1) and (total % 50 == 1):
      sys.stdout.write('\n')
    if output.UnexpectedOutput():

• Read up
  Read up

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

Open

def CheckComment(comment, filename, linenum, error):
  """Checks for common mistakes in TODO comments.

  Args:
    comment: The text of the comment from the line in question.

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

          if not os.path.exists(home_dot_gyp):
            home_dot_gyp = None
          else:
            break
  else:

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

Open

def CheckLanguage(filename, clean_lines, linenum, file_extension, include_state,