Skip to content
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

more linting #174

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

more linting #174

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
17c23f6
initialize max_pos[] with zeros
fperrad Oct 28, 2022
602c0d8
move cast inside macro mqtt_mq_currsz
fperrad Oct 28, 2022
6b0b828
use same parameter name as in prototype
fperrad Oct 28, 2022
8060234
add missing parameter name
fperrad Oct 28, 2022
b258c7f
remove useless mask
fperrad Oct 28, 2022
f2f02c4
remove useless else
fperrad Nov 12, 2022
cf59035
avoid multiple return
fperrad Nov 15, 2022
294e4fd
remove redundant parentheses
fperrad Nov 21, 2022
4a18fe9
refactor types in mqtt_message_queue
fperrad Nov 23, 2022
d84fc67
refactor prototype of mqtt_mq_init
fperrad Nov 21, 2022
40b2b04
remove some checks of buf
fperrad Nov 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
avoid multiple return
  • Loading branch information
@fperrad
fperrad committed Nov 15, 2022
commit cf5903595c0c3e56d079744efda8478fb193d6c9
7 changes: 5 additions & 2 deletions src/mqtt.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1734,9 +1734,12 @@ ssize_t mqtt_unpack_response(struct mqtt_response* response, const uint8_t *buf,
rv = mqtt_unpack_unsuback_response(response, buf);
break;
case MQTT_CONTROL_PINGRESP:
return rv;
/* nothing to unpack */
rv = 0;
break;
default:
return MQTT_ERROR_RESPONSE_INVALID_CONTROL_TYPE;
rv = MQTT_ERROR_RESPONSE_INVALID_CONTROL_TYPE;
break;
}

if (rv < 0) return rv;
Expand Down