Skip to content

Commit 05139f0

Browse files
committed
Make editorconfig less aggressive, don't set any prefs and don't force a pref if it was already explicitly set
1 parent df95d32 commit 05139f0

File tree

1 file changed

+4
-15
lines changed

1 file changed

+4
-15
lines changed

content/editorconfig.js

+4-15
Original file line numberDiff line numberDiff line change
@@ -31,21 +31,10 @@
3131

3232
for (let k in items)
3333
{
34-
switch (typeof items[k])
35-
{
36-
case 'string':
37-
prefs.setStringPref(k, items[k]);
38-
break;
39-
case 'boolean':
40-
prefs.setBooleanPref(k, items[k]);
41-
break;
42-
case 'number':
43-
prefs.setLongPref(k, items[k]);
44-
break;
45-
default:
46-
log.warn("Invalid pref type for: " + k);
47-
break;
48-
}
34+
if (prefs.hasPrefHere(k))
35+
continue; // don't set prefs that are already explicitly set
36+
37+
prefs.prefObserverService.notifyObservers(prefs, k, items[k]);
4938
}
5039
}
5140

0 commit comments

Comments
 (0)