Closed zmstone closed 3 years ago
noticed this while fixing dialyzer warnings in dev/v5.0 branch in emqx project.
It seems to me that the first argument should be a #mqtt_packet{} record. i.e. the code prior to this change had the arguments in the wrong order.
#mqtt_packet{}
I am however not quite sure if it should be clientid from state, or something else from the channel info to be used as the second arg.
clientid
noticed this while fixing dialyzer warnings in dev/v5.0 branch in emqx project.
It seems to me that the first argument should be a
#mqtt_packet{}record. i.e. the code prior to this change had the arguments in the wrong order.I am however not quite sure if it should be
clientidfrom state, or something else from the channel info to be used as the second arg.