jish/pre-commit

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

Open

        if (b) {
            indent += option.indent;
            if (nexttoken.from === indent + option.indent) {
                indent += option.indent;
            }

• Read up
  Read up

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

Open

                                        case Tokens.NAMESPACE_SYM:
                                            token = tokenStream.LT(1);
                                            this._namespace(false);
                                            throw new SyntaxError("@namespace not allowed here.", token.startLine, token.startCol);

• Read up
  Read up

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

Open

                                        case Tokens.CHARSET_SYM:
                                            token = tokenStream.LT(1);
                                            this._charset(false);
                                            throw new SyntaxError("@charset not allowed here.", token.startLine, token.startCol);

• Read up
  Read up

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

Open

                                        case Tokens.IMPORT_SYM:
                                            token = tokenStream.LT(1);
                                            this._import(false);
                                            throw new SyntaxError("@import not allowed here.", token.startLine, token.startCol);

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                        if (temp.length == 1){
                            reader.reset();
                        } else {
                            value += temp;
                        }

Avoid deeply nested control flow statements.

Open

                        if (ValidationTypes.isAny(expression, xDir)) {
                                result = true;
                                ValidationTypes.isAny(expression, numeric);
                        } else if (ValidationTypes.isAny(expression, "center")) {
                            result = true;

Avoid deeply nested control flow statements.

Open

                            if (!lastProperty || (lastProperty.property.text.toLowerCase() != name || lastProperty.colorType != "compat")){
                                reporter.report("Fallback " + name + " (hex or RGB) should precede " + colorType + " " + name + ".", event.line, event.col, rule);
                            }

Avoid deeply nested control flow statements.

Open

                            if (xmode === 'html') {
                                if (option.safe) {
                                    warningAt("ADsafe string violation.",
                                            line, character + j);
                                }

Avoid deeply nested control flow statements.

Open

                        if (left.value === 'Math') {
                            warning("Math is not a function.", left);
                        } else if (option.newcap) {
                            warning(
"Missing 'new' prefix when invoking a constructor.", left);

Avoid deeply nested control flow statements.

Open

                    } else if (a === 'style') {
                        xmode = 'scriptstring';
                        advance('=');
                        q = nexttoken.id;
                        if (q !== '"' && q !== "'") {

Avoid deeply nested control flow statements.

Open

                            if (!(prop == "padding" && value.parts.length === 2 && value.parts[1].value === 0)){
                                reporter.report("Using width with " + prop + " can sometimes make elements larger than you expect.", properties[prop].line, properties[prop].col, rule);
                            }

Avoid deeply nested control flow statements.

Open

                            if (CSSLint.Util.indexOf(actual, item) === -1) {
                                propertiesSpecified = (actual.length === 1) ? actual[0] : (actual.length == 2) ? actual.join(" and ") : actual.join(", ");
                                reporter.report("The property " + item + " is compatible with " + propertiesSpecified + " and should be included as well.", value.actualNodes[0].line, value.actualNodes[0].col, rule);
                            }

Avoid deeply nested control flow statements.

Open

                        if (nexttoken.id !== q) {
                            error("Missing close quote on script attribute.");
                        }

Avoid deeply nested control flow statements.

Open

                                if (!nextLine()) {
                                    errorAt("Unclosed comment.", line, character);
                                } else {
                                    if (option.safe && ax.test(s)) {
                                        warningAt("ADsafe comment violation.", line, character);

Avoid deeply nested control flow statements.

Open

                                for (;;) {
                                    b = true;
                                    c = s.charAt(l);
                                    l += 1;
                                    switch (c) {

Avoid deeply nested control flow statements.

Open

                                if (combinator !== null){
                                    selector.push(combinator);
                                } else {
                                    selector.push(ws);
                                }

Avoid deeply nested control flow statements.

Open

                    if (ValidationTypes.isAny(expression, xDir + " | " + yDir)) {
                        result = true;
                        ValidationTypes.isAny(expression, numeric);
                    }

Avoid deeply nested control flow statements.

Open

                            if (!propertyGroups[prop]) {
                                propertyGroups[prop] = {
                                    full : variations.slice(0),
                                    actual : [],
                                    actualNodes: []

Avoid deeply nested control flow statements.

Open

                        if (!option.on) {
                            warning("Avoid HTML event handlers.");
                        }

Avoid deeply nested control flow statements.

Open

                            if (CSSLint.Util.indexOf(propertyGroups[prop].actual, name.text) === -1) {
                                propertyGroups[prop].actual.push(name.text);
                                propertyGroups[prop].actualNodes.push(name);
                            }