Skip to content

Commit d23bd49

Browse files
committed
fix: sonar issues
1 parent 6240311 commit d23bd49

File tree

3 files changed

+3
-19
lines changed

3 files changed

+3
-19
lines changed

src/extensions/default/CSSCodeHints/main.js

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,6 @@ define(function (require, exports, module) {
2727
var AppInit = brackets.getModule("utils/AppInit"),
2828
CodeHintManager = brackets.getModule("editor/CodeHintManager"),
2929
CSSUtils = brackets.getModule("language/CSSUtils"),
30-
HTMLUtils = brackets.getModule("language/HTMLUtils"),
31-
LanguageManager = brackets.getModule("language/LanguageManager"),
3230
PreferencesManager = brackets.getModule("preferences/PreferencesManager"),
3331
TokenUtils = brackets.getModule("utils/TokenUtils"),
3432
StringMatch = brackets.getModule("utils/StringMatch"),
@@ -234,12 +232,11 @@ define(function (require, exports, module) {
234232
this.cursor = this.editor.getCursorPos();
235233
this.info = CSSUtils.getInfoAtPos(this.editor, this.cursor);
236234

237-
var needle = this.info.name,
235+
let needle = this.info.name,
238236
valueNeedle = "",
239237
context = this.info.context,
240238
valueArray,
241239
type,
242-
namedFlows,
243240
result,
244241
selectInitial = false;
245242

@@ -300,7 +297,7 @@ define(function (require, exports, module) {
300297
});
301298

302299
return {
303-
hints: formatHints(result, valueNeedle),
300+
hints: formatHints(result),
304301
match: null, // the CodeHintManager should not format the results
305302
selectInitial: selectInitial
306303
};
@@ -331,7 +328,7 @@ define(function (require, exports, module) {
331328
});
332329

333330
return {
334-
hints: formatHints(result, needle),
331+
hints: formatHints(result),
335332
match: null, // the CodeHintManager should not format the results
336333
selectInitial: selectInitial,
337334
handleWideResults: false

src/extensions/default/CSSCodeHints/unittests.js

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -110,15 +110,6 @@ define(function (require, exports, module) {
110110
expect(hintList[0]).toBe(expectedFirstHint);
111111
}
112112

113-
// compares lists to ensure they are the same
114-
function verifyListsAreIdentical(hintList, values) {
115-
var i;
116-
expect(hintList.length).toBe(values.length);
117-
for (i = 0; i < values.length; i++) {
118-
expect(hintList[i]).toBe(values[i]);
119-
}
120-
}
121-
122113

123114
function selectHint(provider, expectedHint, implicitChar) {
124115
var hintList = expectHints(provider, implicitChar);

src/language/CSSUtils.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,6 @@ define(function (require, exports, module) {
4343
PROP_VALUE = "prop.value",
4444
IMPORT_URL = "import.url";
4545

46-
var RESERVED_FLOW_NAMES = ["content", "element"],
47-
INVALID_FLOW_NAMES = ["none", "inherit", "default", "auto", "initial"],
48-
IGNORED_FLOW_NAMES = RESERVED_FLOW_NAMES.concat(INVALID_FLOW_NAMES);
49-
5046
/**
5147
* List of all bracket pairs that is keyed by opening brackets, and the inverted list
5248
* that is keyed by closing brackets.

0 commit comments

Comments
 (0)