[Rule] Prefer let-bindings over var-bindings wherever possible about swiftlint HOT 9 CLOSED

Wouldn't you also need to check if any mutating methods /input functions are called with it?

from swiftlint.

Yeah, this will be a trick one. We can't also predict anything on properties from public API (they might not change inside the file itself, but a consumer may change it).

from swiftlint.

An easier way to do this is trying to replace every (non-public?) var with let and let the compiler check if it's still ok.

The problems are that this will probably be much slower and that neither syntax nor structure expose this error (but swiftc does).

from swiftlint.

@mrackwitz is working on a parser for swiftc -dump-ast so hopefully that'll make it easier to build different kinds of SwiftLint rules based on that version of the AST.

from swiftlint.

👍

I was just about to comment about swiftc -dump-ast and how helpful it would be.

from swiftlint.

This is now a compiler warning on Swift 2.0, so maybe it should be closed.

from swiftlint.

This is now a compiler warning on Swift 2.0, so maybe it should be closed.

Agreed. Plus, this would have been difficult to do without LLMV IR access.

from swiftlint.

Any chance of supporting this for class / struct properties? I noticed the swift compiler doesn't catch this. Is it too complex to support?

from swiftlint.

It would be possible to build an analyzer rule to do that.

from swiftlint.