testlint icon indicating copy to clipboard operation
testlint copied to clipboard

False positive on self.

Open chiliec opened this issue 10 years ago • 1 comments

endpoint.update().onComplete { _ in
    let emergency = EmergencyRoomsDataSource(database: self.database)
}

In closure, testlint say I don't need to use self.

chiliec avatar Nov 26 '15 18:11 chiliec

  • For now, annotate with // nwm to skip warning
  • Once Swift goes open source, I may be able to do better testing for closures

-- E

On Nov 26, 2015, at 11:04 AM, Vladimir Babin [email protected] wrote:

endpoint.update().onComplete { _ in let emergency = EmergencyRoomsDataSource(database: self.database) } In closure, testlint say I don't need to use self.

— Reply to this email directly or view it on GitHub https://github.com/erica/testlint/issues/2.

erica avatar Nov 26 '15 18:11 erica