Showing 1,896 of 2,859 total issues

Function ProcessOptions has a Cognitive Complexity of 35 (exceeds 5 allowed). Consider refactoring.
Open

def ProcessOptions(options):
  global VARIANT_FLAGS
  global VARIANTS

  # Architecture and mode related stuff.
Severity: Minor
Found in deps/v8/tools/run-tests.py - About 5 hrs to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function RunFuzzer has a Cognitive Complexity of 35 (exceeds 5 allowed). Consider refactoring.
Open

def RunFuzzer(process_id, options, stop_running):
  MAX_SLEEP_TIME = 0.1
  INITIAL_SLEEP_TIME = 0.001
  SLEEP_TIME_FACTOR = 1.25
  base_file_name = "/dev/shm/runtime_fuzz_%d" % process_id
Severity: Minor
Found in deps/v8/tools/generate-runtime-tests.py - About 5 hrs to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function gunzTarPerm has 131 lines of code (exceeds 25 allowed). Consider refactoring.
Open

function gunzTarPerm (tarball, target, dMode, fMode, uid, gid, cb_) {
  if (!dMode) dMode = npm.modes.exec
  if (!fMode) fMode = npm.modes.file
  log.silly("gunzTarPerm", "modes", [dMode.toString(8), fMode.toString(8)])

Severity: Major
Found in deps/npm/lib/utils/tar.js - About 5 hrs to fix

    File json.js has 382 lines of code (exceeds 250 allowed). Consider refactoring.
    Open

    // Copyright Joyent, Inc. and other Node contributors.
    //
    // Permission is hereby granted, free of charge, to any person obtaining a
    // copy of this software and associated documentation files (the
    // "Software"), to deal in the Software without restriction, including
    Severity: Minor
    Found in tools/doc/json.js - About 5 hrs to fix

      Function gunzTarPerm has a Cognitive Complexity of 34 (exceeds 5 allowed). Consider refactoring.
      Open

      function gunzTarPerm (tarball, target, dMode, fMode, uid, gid, cb_) {
        if (!dMode) dMode = npm.modes.exec
        if (!fMode) fMode = npm.modes.file
        log.silly("gunzTarPerm", "modes", [dMode.toString(8), fMode.toString(8)])
      
      
      Severity: Minor
      Found in deps/npm/lib/utils/tar.js - About 5 hrs to fix

      Cognitive Complexity

      Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

      A method's cognitive complexity is based on a few simple rules:

      • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
      • Code is considered more complex for each "break in the linear flow of the code"
      • Code is considered more complex when "flow breaking structures are nested"

      Further reading

      Function makeEnv has a Cognitive Complexity of 34 (exceeds 5 allowed). Consider refactoring.
      Open

      function makeEnv (data, prefix, env) {
        prefix = prefix || "npm_package_"
        if (!env) {
          env = {}
          for (var i in process.env) if (!i.match(/^npm_/)) {
      Severity: Minor
      Found in deps/npm/lib/utils/lifecycle.js - About 5 hrs to fix

      Cognitive Complexity

      Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

      A method's cognitive complexity is based on a few simple rules:

      • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
      • Code is considered more complex for each "break in the linear flow of the code"
      • Code is considered more complex when "flow breaking structures are nested"

      Further reading

      File run-deopt-fuzzer.py has 377 lines of code (exceeds 250 allowed). Consider refactoring.
      Open

      #!/usr/bin/env python
      #
      # Copyright 2012 the V8 project authors. All rights reserved.
      # Redistribution and use in source and binary forms, with or without
      # modification, are permitted provided that the following conditions are
      Severity: Minor
      Found in deps/v8/tools/run-deopt-fuzzer.py - About 5 hrs to fix

        MsvsSettings has 38 functions (exceeds 20 allowed). Consider refactoring.
        Open

        class MsvsSettings(object):
          """A class that understands the gyp 'msvs_...' values (especially the
          msvs_settings field). They largely correpond to the VS2008 IDE DOM. This
          class helps map those settings to command line options."""
        
        
        Severity: Minor
        Found in tools/gyp/pylib/gyp/msvs_emulation.py - About 5 hrs to fix

          Function MergeLists has a Cognitive Complexity of 33 (exceeds 5 allowed). Consider refactoring.
          Open

          def MergeLists(to, fro, to_file, fro_file, is_paths=False, append=True):
            # Python documentation recommends objects which do not support hash
            # set this value to None. Python library objects follow this rule.
            is_hashable = lambda val: val.__hash__
          
          
          Severity: Minor
          Found in tools/gyp/pylib/gyp/input.py - About 4 hrs to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          Function ProcessContents has a Cognitive Complexity of 33 (exceeds 5 allowed). Consider refactoring.
          Open

            def ProcessContents(self, name, contents):
              result = True
              base = basename(name)
              if not base in SourceProcessor.IGNORE_TABS:
                if '\t' in contents:
          Severity: Minor
          Found in deps/v8/tools/presubmit.py - About 4 hrs to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          Function Execute has a Cognitive Complexity of 33 (exceeds 5 allowed). Consider refactoring.
          Open

          def Execute(arch, mode, args, options, suites, workspace):
            print(">>> Running tests for %s.%s" % (arch, mode))
          
            shell_dir = options.shell_dir
            if not shell_dir:
          Severity: Minor
          Found in deps/v8/tools/run-tests.py - About 4 hrs to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          File add-remote-git.js has 368 lines of code (exceeds 250 allowed). Consider refactoring.
          Open

          var assert = require('assert')
          var crypto = require('crypto')
          var fs = require('graceful-fs')
          var path = require('path')
          var url = require('url')
          Severity: Minor
          Found in deps/npm/lib/cache/add-remote-git.js - About 4 hrs to fix

            File timers.js has 367 lines of code (exceeds 250 allowed). Consider refactoring.
            Open

            // Copyright Joyent, Inc. and other Node contributors.
            //
            // Permission is hereby granted, free of charge, to any person obtaining a
            // copy of this software and associated documentation files (the
            // "Software"), to deal in the Software without restriction, including
            Severity: Minor
            Found in lib/timers.js - About 4 hrs to fix

              File indentation.py has 367 lines of code (exceeds 250 allowed). Consider refactoring.
              Open

              #!/usr/bin/env python
              #
              # Copyright 2010 The Closure Linter Authors. All Rights Reserved.
              #
              # Licensed under the Apache License, Version 2.0 (the "License");
              Severity: Minor
              Found in tools/closure_linter/closure_linter/indentation.py - About 4 hrs to fix

                File presubmit.py has 367 lines of code (exceeds 250 allowed). Consider refactoring.
                Open

                #!/usr/bin/env python
                #
                # Copyright 2012 the V8 project authors. All rights reserved.
                # Redistribution and use in source and binary forms, with or without
                # modification, are permitted provided that the following conditions are
                Severity: Minor
                Found in deps/v8/tools/presubmit.py - About 4 hrs to fix

                  Function formatProperty has a Cognitive Complexity of 32 (exceeds 5 allowed). Consider refactoring.
                  Open

                  function formatProperty(ctx, value, recurseTimes, visibleKeys, key, array) {
                    var name, str, desc;
                    desc = Object.getOwnPropertyDescriptor(value, key) || { value: value[key] };
                    if (desc.get) {
                      if (desc.set) {
                  Severity: Minor
                  Found in lib/util.js - About 4 hrs to fix

                  Cognitive Complexity

                  Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                  A method's cognitive complexity is based on a few simple rules:

                  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                  • Code is considered more complex for each "break in the linear flow of the code"
                  • Code is considered more complex when "flow breaking structures are nested"

                  Further reading

                  Function listen has a Cognitive Complexity of 32 (exceeds 5 allowed). Consider refactoring.
                  Open

                  Server.prototype.listen = function() {
                    var self = this;
                  
                    var lastArg = arguments[arguments.length - 1];
                    if (util.isFunction(lastArg)) {
                  Severity: Minor
                  Found in lib/net.js - About 4 hrs to fix

                  Cognitive Complexity

                  Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                  A method's cognitive complexity is based on a few simple rules:

                  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                  • Code is considered more complex for each "break in the linear flow of the code"
                  • Code is considered more complex when "flow breaking structures are nested"

                  Further reading

                  Function execFile has a Cognitive Complexity of 32 (exceeds 5 allowed). Consider refactoring.
                  Open

                  exports.execFile = function(file /* args, options, callback */) {
                    var args = [], callback;
                    var options = {
                      encoding: 'utf8',
                      timeout: 0,
                  Severity: Minor
                  Found in lib/child_process.js - About 4 hrs to fix

                  Cognitive Complexity

                  Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                  A method's cognitive complexity is based on a few simple rules:

                  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                  • Code is considered more complex for each "break in the linear flow of the code"
                  • Code is considered more complex when "flow breaking structures are nested"

                  Further reading

                  Function Buffer has a Cognitive Complexity of 32 (exceeds 5 allowed). Consider refactoring.
                  Open

                  function Buffer(subject, encoding) {
                    if (!util.isBuffer(this))
                      return new Buffer(subject, encoding);
                  
                    if (util.isNumber(subject)) {
                  Severity: Minor
                  Found in lib/buffer.js - About 4 hrs to fix

                  Cognitive Complexity

                  Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                  A method's cognitive complexity is based on a few simple rules:

                  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                  • Code is considered more complex for each "break in the linear flow of the code"
                  • Code is considered more complex when "flow breaking structures are nested"

                  Further reading

                  Function realpath has a Cognitive Complexity of 32 (exceeds 5 allowed). Consider refactoring.
                  Open

                  fs.realpath = function realpath(p, cache, cb) {
                    if (!util.isFunction(cb)) {
                      cb = maybeCallback(cache);
                      cache = null;
                    }
                  Severity: Minor
                  Found in lib/fs.js - About 4 hrs to fix

                  Cognitive Complexity

                  Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                  A method's cognitive complexity is based on a few simple rules:

                  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                  • Code is considered more complex for each "break in the linear flow of the code"
                  • Code is considered more complex when "flow breaking structures are nested"

                  Further reading

                  Severity
                  Category
                  Status
                  Source
                  Language