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

Show file tree
Hide file tree
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
move cast inside macro mqtt_mq_currsz 
the cast was present 3 times (see function mqtt_mq_register & mqtt_mq_clean)
but missing in function mqtt_mq_init
  • Loading branch information
@fperrad
fperrad committed Oct 28, 2022
commit 602c0d8c9465db56fc8f8a87d64f65729f62a2f0
2 changes: 1 addition & 1 deletion include/mqtt.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1081,7 +1081,7 @@ struct mqtt_queued_message* mqtt_mq_find(const struct mqtt_message_queue *mq, en
* @brief Used internally to recalculate the \c curr_sz.
* @ingroup details
*/
#define mqtt_mq_currsz(mq_ptr) (((mq_ptr)->curr >= (uint8_t*) ((mq_ptr)->queue_tail - 1)) ? 0 : ((uint8_t*) ((mq_ptr)->queue_tail - 1)) - (mq_ptr)->curr)
#define mqtt_mq_currsz(mq_ptr) (((mq_ptr)->curr >= (uint8_t*) ((mq_ptr)->queue_tail - 1)) ? 0 : (size_t) (((uint8_t*) ((mq_ptr)->queue_tail - 1)) - (mq_ptr)->curr))

/* CLIENT */

Expand Down
8 changes: 4 additions & 4 deletions src/mqtt.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1619,7 +1619,7 @@ void mqtt_mq_init(struct mqtt_message_queue *mq, void *buf, size_t bufsz)
mq->mem_end = (uint8_t *)buf + bufsz;
mq->curr = (uint8_t *)buf;
mq->queue_tail = (struct mqtt_queued_message *)mq->mem_end;
mq->curr_sz = buf == NULL ? 0 : mqtt_mq_currsz(mq);
mq->curr_sz = (buf == NULL) ? 0 : mqtt_mq_currsz(mq);
}

struct mqtt_queued_message* mqtt_mq_register(struct mqtt_message_queue *mq, size_t nbytes)
Expand All @@ -1632,7 +1632,7 @@ struct mqtt_queued_message* mqtt_mq_register(struct mqtt_message_queue *mq, size

/* move curr and recalculate curr_sz */
mq->curr += nbytes;
mq->curr_sz = (size_t) (mqtt_mq_currsz(mq));
mq->curr_sz = mqtt_mq_currsz(mq);

return mq->queue_tail;
}
Expand All @@ -1648,7 +1648,7 @@ void mqtt_mq_clean(struct mqtt_message_queue *mq) {
if (new_head < mq->queue_tail) {
mq->curr = (uint8_t *)mq->mem_start;
mq->queue_tail = (struct mqtt_queued_message *)mq->mem_end;
mq->curr_sz = (size_t) (mqtt_mq_currsz(mq));
mq->curr_sz = mqtt_mq_currsz(mq);
return;
} else if (new_head == mqtt_mq_get(mq, 0)) {
/* do nothing */
Expand Down Expand Up @@ -1680,7 +1680,7 @@ void mqtt_mq_clean(struct mqtt_message_queue *mq) {
}

/* get curr_sz */
mq->curr_sz = (size_t) (mqtt_mq_currsz(mq));
mq->curr_sz = mqtt_mq_currsz(mq);
}

struct mqtt_queued_message* mqtt_mq_find(const struct mqtt_message_queue *mq, enum MQTTControlPacketType control_type, const uint16_t *packet_id)
Expand Down