[commit] master: make free_*_messages() loops less obfuscated
mailbox synchronizer
Brought to you by:
ossi
|
From: Oswald B. <os...@us...> - 2022-06-19 14:36:22
|
commit b9f0162642e270d5ebac1021cdac1b1a9fc3dc24
Author: Oswald Buddenhagen <os...@us...>
Date: Thu Jun 2 18:42:07 2022 +0200
make free_*_messages() loops less obfuscated
notably, free_maildir_messages() had a dead assignment.
src/driver.c | 7 +++----
src/drv_maildir.c | 5 +++--
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/driver.c b/src/driver.c
index 6eabb9b3..31066bb3 100644
--- a/src/driver.c
+++ b/src/driver.c
@@ -24,12 +24,11 @@ count_generic_messages( message_t *msgs )
void
free_generic_messages( message_t *msgs )
{
- message_t *tmsg;
-
- for (; msgs; msgs = tmsg) {
- tmsg = msgs->next;
+ while (msgs) {
+ message_t *tmsg = msgs->next;
free( msgs->msgid );
free( msgs );
+ msgs = tmsg;
}
}
diff --git a/src/drv_maildir.c b/src/drv_maildir.c
index 0dce555e..8514d3c2 100644
--- a/src/drv_maildir.c
+++ b/src/drv_maildir.c
@@ -251,11 +251,12 @@ maildir_connect_store( store_t *gctx,
static void
free_maildir_messages( maildir_message_t *msg )
{
- for (maildir_message_t *tmsg; (tmsg = msg); msg = tmsg) {
- tmsg = msg->next;
+ while (msg) {
+ maildir_message_t *tmsg = msg->next;
free( msg->base );
free( msg->msgid );
free( msg );
+ msg = tmsg;
}
}
|