Fix incorrect check (#281)

duckdoom4 · web-flow · commit 7b8bbc9cfb8b · 2023-03-07T08:22:09.000+01:00
!== 0 would always be true now. Let's change it to check if the number of rules has changed, and only update if they are different
diff --git a/src/decorator.ts b/src/decorator.ts
@@ -59,7 +59,7 @@ export function injectCustomTextmateTokens(rules: ValueEqualsSet<TextMateRule>)
         newRules.add(new TextMateRule(rule.scope, rule.settings));
     }
 
-    if (newRules.size !== 0) {
+    if (newRules.size !== currentRules.length) {
         tokenColorCustomizations.textMateRules = newRules.toArray();
         tokensConfig.update("tokenColorCustomizations", tokenColorCustomizations, ConfigurationTarget.Workspace).then(
             () => {

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ export function injectCustomTextmateTokens(rules: ValueEqualsSet<TextMateRule>)`
`59`	`59`	`newRules.add(new TextMateRule(rule.scope, rule.settings));`
`60`	`60`	`}`
`61`	`61`
`62`		`- if (newRules.size !== 0) {`
	`62`	`+ if (newRules.size !== currentRules.length) {`
`63`	`63`	`tokenColorCustomizations.textMateRules = newRules.toArray();`
`64`	`64`	`tokensConfig.update("tokenColorCustomizations", tokenColorCustomizations, ConfigurationTarget.Workspace).then(`
`65`	`65`	`() => {`