False Positive: Subclasses which implement optional protocol methods

[noahsark769]

Consider the following code:

import Foundation

@objc protocol ProtocolWithOptionalMethods: AnyObject {
    func a()
    @objc optional func b()
}

class Superclass: NSObject, ProtocolWithOptionalMethods {
    func a() {
        print("a")
    }
}

class Subclass: Superclass {
    func b() {
        print("b")
    }
}

Currently, Pecker reports function b() was never used; consider removing it, but b is in fact used as an implementation of the optional method. I put together an example project to demonstrate this here: https://github.com/noahsark769/NGPeckerExamples/blob/master/NGPeckerExamples/OptionalFunctionFromSuperclass.swift#L23

[woshiccm]

Coool, will take a look at it

[woshiccm]

Have check it, this case is really hard to deal. This issue will be dealt with as a low priority, first deal with important issues, hope you understand

[noahsark769]

Yes of course! I can imagine it's not easy. Just wanted to report it so that we could track it 👍

[woshiccm]

@noahsark769 #43
This issue had been fixed, only skip function with optional modifier, this means @objc optional func b() skipped, func b() not skip, temporarily unable to do it 😂. But I think it usually enough, because func b() needs to be used elsewhere.

[woshiccm]

And this issue will be closed.