codenautas/self-explain

Function join has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

      function join(aRoot, aPath) {
        if (aRoot === '') {
          aRoot = '.';
        }
        if (aPath === '') {

Function IndexedSourceMapConsumer_parseMappings has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

      IndexedSourceMapConsumer.prototype._parseMappings = function IndexedSourceMapConsumer_parseMappings(aStr, aSourceRoot) {
        this.__generatedMappings = [];
        this.__originalMappings = [];
        for (var i = 0; i < this._sections.length; i++) {
          var section = this._sections[i];

Function VariableDeclaration has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        VariableDeclaration: function (stmt, flags) {
          var result, i, iz, node, bodyFlags, that = this;
          result = [stmt.kind];
          bodyFlags = flags & F_ALLOW_IN ? S_TFFF : S_FFFF;
          function block() {

Function join has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

      function join(left, right) {
        var leftSource, rightSource, leftCharCode, rightCharCode;
        leftSource = toSourceNodeWhenNeeded(left).toString();
        if (leftSource.length === 0) {
          return [right];

Function SourceMapConsumer_eachMapping has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

      SourceMapConsumer.prototype.eachMapping = function SourceMapConsumer_eachMapping(aCallback, aContext, aOrder) {
        var context = aContext || null;
        var order = aOrder || SourceMapConsumer.GENERATED_ORDER;
        var mappings;
        switch (order) {

Function IfStatement has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        IfStatement: function (stmt, flags) {
          var result, bodyFlags, semicolonOptional, that = this;
          withIndent(function () {
            result = [
              'if' + space + '(',

Function define has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

      function define(id, deps, factory) {
        if (Array.isArray(id)) {
          factory = deps;
          deps = id;
          id = undefined;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      module.exports = {
        isKeywordES5: isKeywordES5,
        isKeywordES6: isKeywordES6,
        isReservedWordES5: isReservedWordES5,
        isReservedWordES6: isReservedWordES6,

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      module.exports = {
        isDecimalDigit: isDecimalDigit,
        isHexDigit: isHexDigit,
        isOctalDigit: isOctalDigit,
        isWhiteSpace: isWhiteSpace,

• Read up
  Read up

Function SourceMapConsumer_originalPositionFor has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

      BasicSourceMapConsumer.prototype.originalPositionFor = function SourceMapConsumer_originalPositionFor(aArgs) {
        var needle = {
            generatedLine: util.getArg(aArgs, 'line'),
            generatedColumn: util.getArg(aArgs, 'column')
          };

Consider simplifying this complex logical expression.

Open

        if ((leftCharCode === 43 || leftCharCode === 45) && leftCharCode === rightCharCode || esutils.code.isIdentifierPartES5(leftCharCode) && esutils.code.isIdentifierPartES5(rightCharCode) || leftCharCode === 47 && rightCharCode === 105) {
          return [
            left,
            noEmptySpace(),
            right

Consider simplifying this complex logical expression.

Open

          } else if (!esutils.code.isIdentifierPartES5(code) && (json && code < 32 || !json && !escapeless && (code < 32 || code > 126))) {
            result += escapeAllowedCharacter(code, str.charCodeAt(i + 1));
            continue;
          }

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        if ((leftCharCode === 43 || leftCharCode === 45) && leftCharCode === rightCharCode || esutils.code.isIdentifierPartES5(leftCharCode) && esutils.code.isIdentifierPartES5(rightCharCode) || leftCharCode === 47 && rightCharCode === 105) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

              if ((leftCharCode === 43 || leftCharCode === 45) && leftCharCode === rightCharCode || esutils.code.isIdentifierPartES5(leftCharCode) && esutils.code.isIdentifierPartES5(rightCharCode)) {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        case 6:
          return id === 'return' || id === 'typeof' || id === 'delete' || id === 'switch' || id === 'export' || id === 'import';

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            if(timesDiffer) {
                rv.push(' ');
                rv.push(aas.substr(11, a.getMilliseconds()?12:8));
            }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        case 4:
          return id === 'this' || id === 'else' || id === 'case' || id === 'void' || id === 'with' || id === 'enum';

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            if(timesDiffer) {
                rv.push(' ');
                rv.push(bbs.substr(11, b.getMilliseconds()?12:8));
            }

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                if (ch === 47) {
                  result += '\\';
                } else if (ch === 91) {
                  characterInBrack = true;
                }

Avoid deeply nested control flow statements.

Open

                while ((current2 -= 1) >= 0) {
                  if (!candidate[current2]) {
                    continue;
                  }
                  if (isProperty(nodeType, candidates[current])) {