Oh, in this case IMO warning for 'self.' will never be introduced, it's clear. And as I understand, no support in community for warning for shadowing. OK, so as I understand, the only solution here - tools like SwiftLint and plugins to xcode.

Btw, regarding 'similar to the override keyword', do you mean something like this:

class A {
  public var x = 10

  func f() {
    override var x = 100

    var value: Int? = 1
    if override let value = value {..}
  }
}

?

On 19.05.2016 20:58, David Waite wrote:
I believe the Swift Team has previously stated they do not want opt-in
warnings defining alternate swift ‘grammars’. I don’t believe there are
any existing -W flags for the compiler, for instance.

If there is shadowing warning added, I’m of the opinion it should work
similar to the override keyword.

-DW

On May 19, 2016, at 11:28 AM, Vladimir.S via swift-evolution
<swift-evolution@swift.org> wrote:

On 18.05.2016 22:43, Krystof Vasa wrote:
I agree that shadowing variables is not a good idea, but I stand by
my point that it's potentially dangerous and error-prone to allow
accessing instance variables without `self`.

Could we all agree that we need these options: 1. option to turn on
warnings on accessing instance variables without `self` 2. option to
turn on warnings on shadowing And that by default, none of that
options should be enabled. ?
_______________________________________________ swift-evolution
mailing list swift-evolution@swift.org
https://lists.swift.org/mailman/listinfo/swift-evolution


_______________________________________________
swift-evolution mailing list
swift-evolution@swift.org
https://lists.swift.org/mailman/listinfo/swift-evolution

Reply via email to