adobe/brackets

Avoid deeply nested control flow statements.

Open

                    if ( elem && elem.parentNode ) {
                        // Handle the case where IE, Opera, and Webkit return items
                        // by name instead of ID
                        if ( elem.id === m ) {
                            results.push( elem );

Avoid deeply nested control flow statements.

Open

                            if ( !(unmatched[i] || setMatched[i]) ) {
                                setMatched[i] = pop.call( results );
                            }

Avoid deeply nested control flow statements.

Open

                    if ( !selector ) {
                        push.apply( results, seed );
                        return results;
                    }

Avoid deeply nested control flow statements.

Open

                        if ( (event.result = ret) === false ) {
                            event.preventDefault();
                            event.stopPropagation();
                        }

Avoid deeply nested control flow statements.

Open

                        if ( rscriptType.test( node.type || "" ) &&
                            !data_priv.access( node, "globalEval" ) && jQuery.contains( doc, node ) ) {

                            if ( node.src ) {
                                // Optional AJAX dependency, but won't run scripts if not present

Avoid deeply nested control flow statements.

Open

                    if ( copyIsArray ) {
                        copyIsArray = false;
                        clone = src && jQuery.isArray(src) ? src : [];

                    } else {

Avoid deeply nested control flow statements.

Open

                } else if ( copy !== undefined ) {
                    target[ name ] = copy;
                }

Avoid deeply nested control flow statements.

Open

                        if ( hasScripts ) {
                            // Support: QtWebKit
                            // jQuery.merge because push.apply(_, arraylike) throws
                            jQuery.merge( scripts, getAll( node, "script" ) );
                        }

Avoid deeply nested control flow statements.

Open

                    if ( conv && s[ "throws" ] ) {
                        response = conv( response );
                    } else {
                        try {
                            response = conv( response );

Avoid deeply nested control flow statements.

Open

                    for ( conv2 in converters ) {

                        // If conv2 outputs current
                        tmp = conv2.split( " " );
                        if ( tmp[ 1 ] === current ) {

Avoid deeply nested control flow statements.

Open

                        if ( matches[ sel ] ) {
                            matches.push( handleObj );
                        }

Avoid deeply nested control flow statements.

Open

                                if ( ( ofType ? node.nodeName.toLowerCase() === name : node.nodeType === 1 ) && ++diff ) {
                                    // Cache the index of each encountered element
                                    if ( useCache ) {
                                        (node[ expando ] || (node[ expando ] = {}))[ type ] = [ dirruns, diff ];
                                    }

Avoid deeply nested control flow statements.

Open

          if (options.debug)
            console.error('globstar swallow a segment, and continue')

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

Open

            it("should NOT list prop-name hints right before curly bracket", function () {
                testEditor.setCursorPos({ line: 4, ch: 10 });    // inside .selector, before {
                expectNoHints(CSSCodeHints.cssPropHintProvider);
            });

• Read up
  Read up

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

Open

        function setupTest(content, languageId) {
            var mock = SpecRunnerUtils.createMockEditor(content, languageId);
            testDocument = mock.doc;
            testEditor = mock.editor;
        }

• Read up
  Read up

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

Open

    function addIndicator(id, indicator, visible, style, tooltip, insertBefore) {

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

Open

            it("should list prop-name hints inside multi-line styletags with cursor in first line", function () {
                testEditor.setCursorPos({ line: 9, ch: 18 });   // inside style, after {
                expectHints(CSSCodeHints.cssPropHintProvider);
            });

• Read up
  Read up

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

Open

        function setupTest(content, languageId) {
            var mock = SpecRunnerUtils.createMockEditor(content, languageId);
            testDocument = mock.doc;
            testEditor = mock.editor;
        }

• Read up
  Read up

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

Open

            it("should NOT list prop-name hints right before style value context", function () {
                testEditor.setCursorPos({ line: 4, ch: 11 });    // after =
                expectNoHints(CSSCodeHints.cssPropHintProvider);
            });

• Read up
  Read up

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

Open

            it("should list prop-name hints inside single-line styletags after semicolon", function () {
                testEditor.setCursorPos({ line: 1, ch: 37 });  // inside style, after ;
                expectHints(CSSCodeHints.cssPropHintProvider);
            });

• Read up
  Read up