Add curly brackets to prevent coding errors

This change shuts the "suspicious semicolon" warning.
Signed-off-by: 's avatarGuilherme Maciel Ferreira <guilherme.maciel.ferreira@gmail.com>
parent b9dbd8f7
...@@ -798,7 +798,9 @@ int MQTTAsync_restoreCommands(MQTTAsyncs* client) ...@@ -798,7 +798,9 @@ int MQTTAsync_restoreCommands(MQTTAsyncs* client)
int buflen; int buflen;
if (strncmp(msgkeys[i], PERSISTENCE_COMMAND_KEY, strlen(PERSISTENCE_COMMAND_KEY)) != 0) if (strncmp(msgkeys[i], PERSISTENCE_COMMAND_KEY, strlen(PERSISTENCE_COMMAND_KEY)) != 0)
{
; ;
}
else if ((rc = c->persistence->pget(c->phandle, msgkeys[i], &buffer, &buflen)) == 0) else if ((rc = c->persistence->pget(c->phandle, msgkeys[i], &buffer, &buflen)) == 0)
{ {
MQTTAsync_queuedCommand* cmd = MQTTAsync_restoreCommand(buffer, buflen); MQTTAsync_queuedCommand* cmd = MQTTAsync_restoreCommand(buffer, buflen);
...@@ -1114,7 +1116,9 @@ int MQTTAsync_processCommand(void) ...@@ -1114,7 +1116,9 @@ int MQTTAsync_processCommand(void)
{ {
if ((cmd->command.type == PUBLISH || cmd->command.type == SUBSCRIBE || cmd->command.type == UNSUBSCRIBE) && if ((cmd->command.type == PUBLISH || cmd->command.type == SUBSCRIBE || cmd->command.type == UNSUBSCRIBE) &&
cmd->client->c->outboundMsgs->count >= MAX_MSG_ID - 1) cmd->client->c->outboundMsgs->count >= MAX_MSG_ID - 1)
{
; /* no more message ids available */ ; /* no more message ids available */
}
else else
{ {
command = cmd; command = cmd;
......
...@@ -187,9 +187,13 @@ int MQTTPersistence_restore(Clients *c) ...@@ -187,9 +187,13 @@ int MQTTPersistence_restore(Clients *c)
while (rc == 0 && i < nkeys) while (rc == 0 && i < nkeys)
{ {
if (strncmp(msgkeys[i], PERSISTENCE_COMMAND_KEY, strlen(PERSISTENCE_COMMAND_KEY)) == 0) if (strncmp(msgkeys[i], PERSISTENCE_COMMAND_KEY, strlen(PERSISTENCE_COMMAND_KEY)) == 0)
{
; ;
}
else if (strncmp(msgkeys[i], PERSISTENCE_QUEUE_KEY, strlen(PERSISTENCE_QUEUE_KEY)) == 0) else if (strncmp(msgkeys[i], PERSISTENCE_QUEUE_KEY, strlen(PERSISTENCE_QUEUE_KEY)) == 0)
{
; ;
}
else if ((rc = c->persistence->pget(c->phandle, msgkeys[i], &buffer, &buflen)) == 0) else if ((rc = c->persistence->pget(c->phandle, msgkeys[i], &buffer, &buflen)) == 0)
{ {
MQTTPacket* pack = MQTTPersistence_restorePacket(buffer, buflen); MQTTPacket* pack = MQTTPersistence_restorePacket(buffer, buflen);
...@@ -618,7 +622,9 @@ int MQTTPersistence_restoreMessageQueue(Clients* c) ...@@ -618,7 +622,9 @@ int MQTTPersistence_restoreMessageQueue(Clients* c)
int buflen; int buflen;
if (strncmp(msgkeys[i], PERSISTENCE_QUEUE_KEY, strlen(PERSISTENCE_QUEUE_KEY)) != 0) if (strncmp(msgkeys[i], PERSISTENCE_QUEUE_KEY, strlen(PERSISTENCE_QUEUE_KEY)) != 0)
{
; ;
}
else if ((rc = c->persistence->pget(c->phandle, msgkeys[i], &buffer, &buflen)) == 0) else if ((rc = c->persistence->pget(c->phandle, msgkeys[i], &buffer, &buflen)) == 0)
{ {
MQTTPersistence_qEntry* qe = MQTTPersistence_restoreQueueEntry(buffer, buflen); MQTTPersistence_qEntry* qe = MQTTPersistence_restoreQueueEntry(buffer, buflen);
...@@ -633,7 +639,9 @@ int MQTTPersistence_restoreMessageQueue(Clients* c) ...@@ -633,7 +639,9 @@ int MQTTPersistence_restoreMessageQueue(Clients* c)
} }
} }
if (msgkeys[i]) if (msgkeys[i])
{
free(msgkeys[i]); free(msgkeys[i]);
}
i++; i++;
} }
if (msgkeys != NULL) if (msgkeys != NULL)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment