RUFF C408 rule (unnecessary-collection-call)

[bigcat88]

This rule checks for unnecessary dict(), list() or tuple() calls that can be rewritten as empty literals.

Why is this bad?

It's unnecessary to call, e.g., dict() as opposed to using an empty literal ({}). The former is slower because the name dict must be looked up in the global scope in case it has been rebound.

Examples:

dict()
dict(a=1, b=2)
list()
tuple()

Use instead:

{}
{"a": 1, "b": 2}
[]
()

Ruff's docs link

[bigcat88]

@comfyanonymous should I open a PRs for multiple-starts-ends-with (PIE810) or/and unnecessary-range-start (PIE808) rules, after merge of this one?

They are both will be small, without many edits.

[bigcat88]

just a friendly note: This PR is rebased and ready. If it is not needed, please close it.

[wibur0620]

👌

[bigcat88]

In total, there are more than 100 PRs hanging in the repository, so I close mine.