changeset 22080:4b25a65f8aa1

lib-storage: Explicitly name function pointers in mailbox lists This allows adding more fields to struct mailbox_list_vfuncs without compiler warnings.
author Timo Sirainen <timo.sirainen@dovecot.fi>
date Wed, 17 May 2017 18:13:09 +0300
parents bb8eb229d0fb
children c5eae96dcd71
files src/lib-storage/index/imapc/imapc-list.c src/lib-storage/index/shared/shared-list.c src/lib-storage/list/mailbox-list-fs.c src/lib-storage/list/mailbox-list-index-backend.c src/lib-storage/list/mailbox-list-maildir.c src/lib-storage/list/mailbox-list-none.c
diffstat 6 files changed, 128 insertions(+), 160 deletions(-) [+]
line wrap: on
line diff
--- a/src/lib-storage/index/imapc/imapc-list.c	Wed May 17 17:49:25 2017 +0300
+++ b/src/lib-storage/index/imapc/imapc-list.c	Wed May 17 18:13:09 2017 +0300
@@ -964,27 +964,23 @@
 	.mailbox_name_max_length = MAILBOX_LIST_NAME_MAX_LENGTH,
 
 	.v = {
-		imapc_list_alloc,
-		imapc_list_init,
-		imapc_list_deinit,
-		NULL,
-		imapc_list_get_hierarchy_sep,
-		imapc_list_get_vname,
-		imapc_list_get_storage_name,
-		imapc_list_get_path,
-		imapc_list_get_temp_prefix,
-		imapc_list_join_refpattern,
-		imapc_list_iter_init,
-		imapc_list_iter_next,
-		imapc_list_iter_deinit,
-		NULL,
-		NULL,
-		imapc_list_subscriptions_refresh,
-		imapc_list_set_subscribed,
-		imapc_list_delete_mailbox,
-		imapc_list_delete_dir,
-		imapc_list_delete_symlink,
-		imapc_list_rename_mailbox,
-		NULL, NULL, NULL, NULL
+		.alloc = imapc_list_alloc,
+		.init = imapc_list_init,
+		.deinit = imapc_list_deinit,
+		.get_hierarchy_sep = imapc_list_get_hierarchy_sep,
+		.get_vname = imapc_list_get_vname,
+		.get_storage_name = imapc_list_get_storage_name,
+		.get_path = imapc_list_get_path,
+		.get_temp_prefix = imapc_list_get_temp_prefix,
+		.join_refpattern = imapc_list_join_refpattern,
+		.iter_init = imapc_list_iter_init,
+		.iter_next = imapc_list_iter_next,
+		.iter_deinit = imapc_list_iter_deinit,
+		.subscriptions_refresh = imapc_list_subscriptions_refresh,
+		.set_subscribed = imapc_list_set_subscribed,
+		.delete_mailbox = imapc_list_delete_mailbox,
+		.delete_dir = imapc_list_delete_dir,
+		.delete_symlink = imapc_list_delete_symlink,
+		.rename_mailbox = imapc_list_rename_mailbox,
 	}
 };
--- a/src/lib-storage/index/shared/shared-list.c	Wed May 17 17:49:25 2017 +0300
+++ b/src/lib-storage/index/shared/shared-list.c	Wed May 17 18:13:09 2017 +0300
@@ -288,28 +288,24 @@
 	.props = 0,
 	.mailbox_name_max_length = MAILBOX_LIST_NAME_MAX_LENGTH,
 
-	{
-		shared_list_alloc,
-		NULL,
-		shared_list_deinit,
-		shared_get_storage,
-		shared_list_get_hierarchy_sep,
-		mailbox_list_default_get_vname,
-		mailbox_list_default_get_storage_name,
-		shared_list_get_path,
-		shared_list_get_temp_prefix,
-		shared_list_join_refpattern,
-		shared_list_iter_init,
-		shared_list_iter_next,
-		shared_list_iter_deinit,
-		NULL,
-		NULL,
-		shared_list_subscriptions_refresh,
-		shared_list_set_subscribed,
-		shared_list_delete_mailbox,
-		shared_list_delete_dir,
-		shared_list_delete_symlink,
-		shared_list_rename_mailbox,
-		NULL, NULL, NULL, NULL
+	.v = {
+		.alloc = shared_list_alloc,
+		.deinit = shared_list_deinit,
+		.get_storage = shared_get_storage,
+		.get_hierarchy_sep = shared_list_get_hierarchy_sep,
+		.get_vname = mailbox_list_default_get_vname,
+		.get_storage_name = mailbox_list_default_get_storage_name,
+		.get_path = shared_list_get_path,
+		.get_temp_prefix = shared_list_get_temp_prefix,
+		.join_refpattern = shared_list_join_refpattern,
+		.iter_init = shared_list_iter_init,
+		.iter_next = shared_list_iter_next,
+		.iter_deinit = shared_list_iter_deinit,
+		.subscriptions_refresh = shared_list_subscriptions_refresh,
+		.set_subscribed = shared_list_set_subscribed,
+		.delete_mailbox = shared_list_delete_mailbox,
+		.delete_dir = shared_list_delete_dir,
+		.delete_symlink = shared_list_delete_symlink,
+		.rename_mailbox = shared_list_rename_mailbox,
 	}
 };
--- a/src/lib-storage/list/mailbox-list-fs.c	Wed May 17 17:49:25 2017 +0300
+++ b/src/lib-storage/list/mailbox-list-fs.c	Wed May 17 18:13:09 2017 +0300
@@ -504,28 +504,24 @@
 	.props = 0,
 	.mailbox_name_max_length = MAILBOX_LIST_NAME_MAX_LENGTH,
 
-	{
-		fs_list_alloc,
-		NULL,
-		fs_list_deinit,
-		NULL,
-		fs_list_get_hierarchy_sep,
-		mailbox_list_default_get_vname,
-		mailbox_list_default_get_storage_name,
-		fs_list_get_path,
-		fs_list_get_temp_prefix,
-		fs_list_join_refpattern,
-		fs_list_iter_init,
-		fs_list_iter_next,
-		fs_list_iter_deinit,
-		fs_list_get_mailbox_flags,
-		NULL,
-		mailbox_list_subscriptions_refresh,
-		fs_list_set_subscribed,
-		fs_list_delete_mailbox,
-		fs_list_delete_dir,
-		mailbox_list_delete_symlink_default,
-		fs_list_rename_mailbox,
-		NULL, NULL, NULL, NULL
+	.v = {
+		.alloc = fs_list_alloc,
+		.deinit = fs_list_deinit,
+		.get_hierarchy_sep = fs_list_get_hierarchy_sep,
+		.get_vname = mailbox_list_default_get_vname,
+		.get_storage_name = mailbox_list_default_get_storage_name,
+		.get_path = fs_list_get_path,
+		.get_temp_prefix = fs_list_get_temp_prefix,
+		.join_refpattern = fs_list_join_refpattern,
+		.iter_init = fs_list_iter_init,
+		.iter_next = fs_list_iter_next,
+		.iter_deinit = fs_list_iter_deinit,
+		.get_mailbox_flags = fs_list_get_mailbox_flags,
+		.subscriptions_refresh = mailbox_list_subscriptions_refresh,
+		.set_subscribed = fs_list_set_subscribed,
+		.delete_mailbox = fs_list_delete_mailbox,
+		.delete_dir = fs_list_delete_dir,
+		.delete_symlink = mailbox_list_delete_symlink_default,
+		.rename_mailbox = fs_list_rename_mailbox,
 	}
 };
--- a/src/lib-storage/list/mailbox-list-index-backend.c	Wed May 17 17:49:25 2017 +0300
+++ b/src/lib-storage/list/mailbox-list-index-backend.c	Wed May 17 18:13:09 2017 +0300
@@ -823,29 +823,24 @@
 	.props = MAILBOX_LIST_PROP_NO_ROOT,
 	.mailbox_name_max_length = MAILBOX_LIST_NAME_MAX_LENGTH,
 
-	{
-		index_list_alloc,
-		index_list_init,
-		index_list_deinit,
-		NULL,
-		index_list_get_hierarchy_sep,
-		mailbox_list_default_get_vname,
-		mailbox_list_default_get_storage_name,
-		index_list_get_path,
-		index_list_get_temp_prefix,
-		NULL,
-		index_list_iter_init,
-		index_list_iter_next,
-		index_list_iter_deinit,
-		NULL,
-		NULL,
-		mailbox_list_subscriptions_refresh,
-		index_list_set_subscribed,
-		index_list_delete_mailbox,
-		index_list_delete_dir,
-		index_list_delete_symlink,
-		index_list_rename_mailbox,
-		NULL, NULL, NULL, NULL
+	.v = {
+		.alloc = index_list_alloc,
+		.init = index_list_init,
+		.deinit = index_list_deinit,
+		.get_hierarchy_sep = index_list_get_hierarchy_sep,
+		.get_vname = mailbox_list_default_get_vname,
+		.get_storage_name = mailbox_list_default_get_storage_name,
+		.get_path = index_list_get_path,
+		.get_temp_prefix = index_list_get_temp_prefix,
+		.iter_init = index_list_iter_init,
+		.iter_next = index_list_iter_next,
+		.iter_deinit = index_list_iter_deinit,
+		.subscriptions_refresh = mailbox_list_subscriptions_refresh,
+		.set_subscribed = index_list_set_subscribed,
+		.delete_mailbox = index_list_delete_mailbox,
+		.delete_dir = index_list_delete_dir,
+		.delete_symlink = index_list_delete_symlink,
+		.rename_mailbox = index_list_rename_mailbox,
 	}
 };
 
--- a/src/lib-storage/list/mailbox-list-maildir.c	Wed May 17 17:49:25 2017 +0300
+++ b/src/lib-storage/list/mailbox-list-maildir.c	Wed May 17 18:13:09 2017 +0300
@@ -482,29 +482,24 @@
 		MAILBOX_LIST_PROP_NO_NOSELECT,
 	.mailbox_name_max_length = MAILBOX_LIST_NAME_MAX_LENGTH,
 
-	{
-		maildir_list_alloc,
-		NULL,
-		maildir_list_deinit,
-		NULL,
-		maildir_list_get_hierarchy_sep,
-		mailbox_list_default_get_vname,
-		mailbox_list_default_get_storage_name,
-		maildir_list_get_path,
-		maildir_list_get_temp_prefix,
-		NULL,
-		maildir_list_iter_init,
-		maildir_list_iter_next,
-		maildir_list_iter_deinit,
-		maildir_list_get_mailbox_flags,
-		NULL,
-		mailbox_list_subscriptions_refresh,
-		maildir_list_set_subscribed,
-		maildir_list_delete_mailbox,
-		maildir_list_delete_dir,
-		mailbox_list_delete_symlink_default,
-		maildir_list_rename_mailbox,
-		NULL, NULL, NULL, NULL
+	.v = {
+		.alloc = maildir_list_alloc,
+		.deinit = maildir_list_deinit,
+		.get_hierarchy_sep = maildir_list_get_hierarchy_sep,
+		.get_vname = mailbox_list_default_get_vname,
+		.get_storage_name = mailbox_list_default_get_storage_name,
+		.get_path = maildir_list_get_path,
+		.get_temp_prefix = maildir_list_get_temp_prefix,
+		.iter_init = maildir_list_iter_init,
+		.iter_next = maildir_list_iter_next,
+		.iter_deinit = maildir_list_iter_deinit,
+		.get_mailbox_flags = maildir_list_get_mailbox_flags,
+		.subscriptions_refresh = mailbox_list_subscriptions_refresh,
+		.set_subscribed = maildir_list_set_subscribed,
+		.delete_mailbox = maildir_list_delete_mailbox,
+		.delete_dir = maildir_list_delete_dir,
+		.delete_symlink = mailbox_list_delete_symlink_default,
+		.rename_mailbox = maildir_list_rename_mailbox,
 	}
 };
 
@@ -515,28 +510,23 @@
 		MAILBOX_LIST_PROP_NO_NOSELECT,
 	.mailbox_name_max_length = MAILBOX_LIST_NAME_MAX_LENGTH,
 
-	{
-		imapdir_list_alloc,
-		NULL,
-		maildir_list_deinit,
-		NULL,
-		maildir_list_get_hierarchy_sep,
-		mailbox_list_default_get_vname,
-		mailbox_list_default_get_storage_name,
-		maildir_list_get_path,
-		maildir_list_get_temp_prefix,
-		NULL,
-		maildir_list_iter_init,
-		maildir_list_iter_next,
-		maildir_list_iter_deinit,
-		maildir_list_get_mailbox_flags,
-		NULL,
-		mailbox_list_subscriptions_refresh,
-		maildir_list_set_subscribed,
-		maildir_list_delete_mailbox,
-		maildir_list_delete_dir,
-		mailbox_list_delete_symlink_default,
-		maildir_list_rename_mailbox,
-		NULL, NULL, NULL, NULL
+	.v = {
+		.alloc = imapdir_list_alloc,
+		.deinit = maildir_list_deinit,
+		.get_hierarchy_sep = maildir_list_get_hierarchy_sep,
+		.get_vname = mailbox_list_default_get_vname,
+		.get_storage_name = mailbox_list_default_get_storage_name,
+		.get_path = maildir_list_get_path,
+		.get_temp_prefix = maildir_list_get_temp_prefix,
+		.iter_init = maildir_list_iter_init,
+		.iter_next = maildir_list_iter_next,
+		.iter_deinit = maildir_list_iter_deinit,
+		.get_mailbox_flags = maildir_list_get_mailbox_flags,
+		.subscriptions_refresh = mailbox_list_subscriptions_refresh,
+		.set_subscribed = maildir_list_set_subscribed,
+		.delete_mailbox = maildir_list_delete_mailbox,
+		.delete_dir = maildir_list_delete_dir,
+		.delete_symlink = mailbox_list_delete_symlink_default,
+		.rename_mailbox = maildir_list_rename_mailbox,
 	}
 };
--- a/src/lib-storage/list/mailbox-list-none.c	Wed May 17 17:49:25 2017 +0300
+++ b/src/lib-storage/list/mailbox-list-none.c	Wed May 17 18:13:09 2017 +0300
@@ -155,28 +155,23 @@
 	.props = MAILBOX_LIST_PROP_NO_ROOT,
 	.mailbox_name_max_length = MAILBOX_LIST_NAME_MAX_LENGTH,
 
-	{
-		none_list_alloc,
-		NULL,
-		none_list_deinit,
-		NULL,
-		none_list_get_hierarchy_sep,
-		mailbox_list_default_get_vname,
-		mailbox_list_default_get_storage_name,
-		none_list_get_path,
-		none_list_get_temp_prefix,
-		NULL,
-		none_list_iter_init,
-		none_list_iter_next,
-		none_list_iter_deinit,
-		none_list_get_mailbox_flags,
-		NULL,
-		none_list_subscriptions_refresh,
-		none_list_set_subscribed,
-		none_list_delete_mailbox,
-		none_list_delete_dir,
-		none_list_delete_dir,
-		none_list_rename_mailbox,
-		NULL, NULL, NULL, NULL
+	.v = {
+		.alloc = none_list_alloc,
+		.deinit = none_list_deinit,
+		.get_hierarchy_sep = none_list_get_hierarchy_sep,
+		.get_vname = mailbox_list_default_get_vname,
+		.get_storage_name = mailbox_list_default_get_storage_name,
+		.get_path = none_list_get_path,
+		.get_temp_prefix = none_list_get_temp_prefix,
+		.iter_init = none_list_iter_init,
+		.iter_next = none_list_iter_next,
+		.iter_deinit = none_list_iter_deinit,
+		.get_mailbox_flags = none_list_get_mailbox_flags,
+		.subscriptions_refresh = none_list_subscriptions_refresh,
+		.set_subscribed = none_list_set_subscribed,
+		.delete_mailbox = none_list_delete_mailbox,
+		.delete_dir = none_list_delete_dir,
+		.delete_symlink = none_list_delete_dir,
+		.rename_mailbox = none_list_rename_mailbox,
 	}
 };