New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update CocoaMQTT.swift for make properties thread safe #603

Open

dongtsai wants to merge 1 commit into emqx:master from dongtsai:make-properties-thread-safe

+3 −3

dongtsai commented Jul 16, 2024

In CocoaMQTT.swift, these properties below are not thread safe, this is similar with #543
I use ThreadSafeDictionary make them thread safe.

    public var subscriptions: [String: CocoaMQTTQoS] = [:]

    fileprivate var subscriptionsWaitingAck: [UInt16: [(String, CocoaMQTTQoS)]] = [:]
    fileprivate var unsubscriptionsWaitingAck: [UInt16: [String]] = [:]


          Update CocoaMQTT.swift

9a4b459

marke properties thread safe

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet