fix(no-unused-class-name): detect duplicated class names

[tbashiyy]

close #1259

[changeset-bot]

🦋 Changeset detected

Latest commit: ffc2030

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
eslint-plugin-svelte	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

Try the Instant Preview in Online Playground

ESLint Online Playground

Install the Instant Preview to Your Local

npm i https://pkg.pr.new/eslint-plugin-svelte@ffc2030

Published Instant Preview Packages:

• eslint-plugin-svelte: https://pkg.pr.new/eslint-plugin-svelte@ffc2030

View Commit

[baseballyama]

Is this better in terms of performance?

Suggested change

	const classesUsedInTemplate: {
	className: string;
	loc: AST.SourceLocation;
	}[] = [];
	const classesUsedInTemplate: Record<string, AST.SourceLocation[] = {}

[tbashiyy]

I think performance is no changes for now.
Because this code just execute by loop. (No access by key.)

If data structure is changed, I think code will be complex a little for now.