adobe/brackets

Function getHints has 68 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    SVGCodeHints.prototype.getHints = function (implicitChar) {
        var hints = [], query, tagInfo, attributes = [], options = [], index, isMultiple, tagSpecificOptions;

        tagInfo  = XMLUtils.getTagInfo(this.editor, this.editor.getCursorPos());
        this.tagInfo = tagInfo;

Function insertHint has 59 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    SVGCodeHints.prototype.insertHint = function (completion) {
        var tagInfo = this.tagInfo,
            pos     = this.editor.getCursorPos(),
            start   = {line: -1, ch: -1},
            end     = {line: -1, ch: -1},

Avoid deeply nested control flow statements.

Open

                if (tagSpecificOptions) {
                    options = tagSpecificOptions.attribOptions;
                    isMultiple = tagSpecificOptions.multiple;
                } else if (attributeData[tagInfo.attrName]) {
                    options = attributeData[tagInfo.attrName].attribOptions;

Avoid deeply nested control flow statements.

Open

                if (!isMultiple && /\s+/.test(tagInfo.token.string)) {
                    return null;
                }

Avoid deeply nested control flow statements.

Open

                    if (tagInfo.exclusionList.indexOf(option) === -1) {
                        var match = StringMatch.stringMatch(option.text || option, query, stringMatcherOptions);
                        if (match) {
                            if (option.color) {
                                match.color = option.color;

Avoid deeply nested control flow statements.

Open

                        if (match) {
                            return match;
                        }

Avoid deeply nested control flow statements.

Open

                if (!tagData.tags[tagInfo.tagName] && !(attributeData[tagInfo.attrName] || tagSpecificOptions)) {
                    return null;
                }

Avoid too many return statements within this function.

Open

        return null;

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

Open

    function formatHints(hints, query) {
        var hasColorSwatch = hints.some(function (token) {
            return token.color;
        });

• Read up
  Read up

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

Open

                    if (tagInfo.exclusionList.indexOf(option) === -1) {
                        var match = StringMatch.stringMatch(option.text || option, query, stringMatcherOptions);
                        if (match) {
                            if (option.color) {
                                match.color = option.color;

• Read up
  Read up