Skip to content

Make PacketProcessor safe without prior SetMsgHandler #211

@jclark

Description

@jclark

Follow-on cleanup from issue #210

Initialise mh to DefaultHandler{} internally in each PacketProcessor so it is never nil. Remove mh != nil guards in dispatch paths (ubx Dispatch lines 156/212, ubx flushNavEpoch line 87, casic, as). Update PacketProcessor interface comments to reflect the new contract: SetMsgHandler is optional; processors are safe to use without it. Add tests to every packet processor that they can process packets with no prior SetMsgHandler call. Once verified, remove the DefaultHandler install from satpulsetool (the internal default suffices).

Metadata

Metadata

Assignees

No one assigned

    Labels

    gpsRelates to gps subsystemin progressCurrently being worked on

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions