Move extension check out of function

Brian King · Brian King · commit a75f61608403 · 2019-01-30T15:03:14.000-05:00
diff --git a/test/decl/var/usage.swift b/test/decl/var/usage.swift
@@ -303,14 +303,6 @@ func sr964() {
       }
     }
   }
-  extension MemberGetterClass {
-    var suspiciousSetterExt: String {
-      get { return "" }
-      set {
-        print(suspiciousSetterExt) // expected-warning {{setter argument 'newValue' was never used, but the property was accessed}} expected-note {{did you mean to use 'newValue' instead of accessing the property's current value?}} {{15-34=newValue}}
-      }
-    }
-  }
   var namedSuspiciousSetter: String {
     get { return "" }
     set(parameter) {
@@ -329,3 +321,12 @@ func sr964() {
     }
   }
 }
+struct MemberGetterExtension {}
+extension MemberGetterExtension {
+  var suspiciousSetter: String {
+    get { return "" }
+    set {
+      print(suspiciousSetter) // expected-warning {{setter argument 'newValue' was never used, but the property was accessed}} expected-note {{did you mean to use 'newValue' instead of accessing the property's current value?}} {{13-29=newValue}}
+    }
+  }
+}

Original file line number	Diff line number	Diff line change
`@@ -303,14 +303,6 @@ func sr964() {`
`303`	`303`	`}`
`304`	`304`	`}`
`305`	`305`	`}`
`306`		`- extension MemberGetterClass {`
`307`		`- var suspiciousSetterExt: String {`
`308`		`- get { return "" }`
`309`		`- set {`
`310`		`- print(suspiciousSetterExt) // expected-warning {{setter argument 'newValue' was never used, but the property was accessed}} expected-note {{did you mean to use 'newValue' instead of accessing the property's current value?}} {{15-34=newValue}}`
`311`		`- }`
`312`		`- }`
`313`		`- }`
`314`	`306`	`var namedSuspiciousSetter: String {`
`315`	`307`	`get { return "" }`
`316`	`308`	`set(parameter) {`
`@@ -329,3 +321,12 @@ func sr964() {`
`329`	`321`	`}`
`330`	`322`	`}`
`331`	`323`	`}`
	`324`	`+struct MemberGetterExtension {}`
	`325`	`+extension MemberGetterExtension {`
	`326`	`+ var suspiciousSetter: String {`
	`327`	`+ get { return "" }`
	`328`	`+ set {`
	`329`	`+ print(suspiciousSetter) // expected-warning {{setter argument 'newValue' was never used, but the property was accessed}} expected-note {{did you mean to use 'newValue' instead of accessing the property's current value?}} {{13-29=newValue}}`
	`330`	`+ }`
	`331`	`+ }`
	`332`	`+}`