From: Stephan Bosch <stephan.bosch@dovecot.fi>
Date: Sun, 23 Oct 2016 15:27:07 +0000 (+0200)
Subject: lib-imap: imap-bodystructure: Grouped together functions relating to the conversion... 
X-Git-Tag: 2.3.0.rc1~2231
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=341c6814c806db208bc4e35131c204b68122b447;p=thirdparty%2Fdovecot%2Fcore.git

lib-imap: imap-bodystructure: Grouped together functions relating to the conversion of BODYSTRUCTURE to BODY.

This only moves code.
---

diff --git a/src/lib-imap/imap-bodystructure.c b/src/lib-imap/imap-bodystructure.c
index bc7e1500ff..da27482f6c 100644
--- a/src/lib-imap/imap-bodystructure.c
+++ b/src/lib-imap/imap-bodystructure.c
@@ -523,74 +523,6 @@ void imap_bodystructure_write(const struct message_part *part,
 		part_write_body(part, dest, extended);
 }
 
-static bool str_append_nstring(string_t *str, const struct imap_arg *arg)
-{
-	const char *cstr;
-
-	if (!imap_arg_get_nstring(arg, &cstr))
-		return FALSE;
-
-	switch (arg->type) {
-	case IMAP_ARG_NIL:
-		str_append(str, "NIL");
-		break;
-	case IMAP_ARG_ATOM:
-		str_append(str, cstr);
-		break;
-	case IMAP_ARG_STRING:
-		str_append_c(str, '"');
-		/* NOTE: we're parsing with no-unescape flag,
-		   so don't double-escape it here */
-		str_append(str, cstr);
-		str_append_c(str, '"');
-		break;
-	case IMAP_ARG_LITERAL: {
-		str_printfa(str, "{%"PRIuSIZE_T"}\r\n", strlen(cstr));
-		str_append(str, cstr);
-		break;
-	}
-	default:
-		i_unreached();
-		return FALSE;
-	}
-	return TRUE;
-}
-
-static void
-imap_write_envelope_list(const struct imap_arg *args, string_t *str,
-	bool toplevel)
-{
-	const struct imap_arg *children;
-
-	/* don't do any typechecking, just write it out */
-	while (!IMAP_ARG_IS_EOL(args)) {
-		bool list = FALSE;
-
-		if (!str_append_nstring(str, args)) {
-			if (!imap_arg_get_list(args, &children)) {
-				/* everything is either nstring or list */
-				i_unreached();
-			}
-
-			str_append_c(str, '(');
-			imap_write_envelope_list(children, str, FALSE);
-			str_append_c(str, ')');
-
-			list = TRUE;
-		}
-		args++;
-
-		if ((toplevel || !list) && !IMAP_ARG_IS_EOL(args))
-			str_append_c(str, ' ');
-	}
-}
-
-static void
-imap_write_envelope(const struct imap_arg *args, string_t *str)
-{
-	imap_write_envelope_list(args, str, TRUE);
-}
-
 static int
 imap_bodystructure_strlist_parse(const struct imap_arg *arg,
 	pool_t pool, const char *const **list_r)
@@ -935,6 +867,74 @@ int imap_bodystructure_parse(const char *bodystructure,
 	return ret;
 }
 
+static bool str_append_nstring(string_t *str, const struct imap_arg *arg)
+{
+	const char *cstr;
+
+	if (!imap_arg_get_nstring(arg, &cstr))
+		return FALSE;
+
+	switch (arg->type) {
+	case IMAP_ARG_NIL:
+		str_append(str, "NIL");
+		break;
+	case IMAP_ARG_ATOM:
+		str_append(str, cstr);
+		break;
+	case IMAP_ARG_STRING:
+		str_append_c(str, '"');
+		/* NOTE: we're parsing with no-unescape flag,
+		   so don't double-escape it here */
+		str_append(str, cstr);
+		str_append_c(str, '"');
+		break;
+	case IMAP_ARG_LITERAL: {
+		str_printfa(str, "{%"PRIuSIZE_T"}\r\n", strlen(cstr));
+		str_append(str, cstr);
+		break;
+	}
+	default:
+		i_unreached();
+		return FALSE;
+	}
+	return TRUE;
+}
+
+static void
+imap_write_envelope_list(const struct imap_arg *args, string_t *str,
+	bool toplevel)
+{
+	const struct imap_arg *children;
+
+	/* don't do any typechecking, just write it out */
+	while (!IMAP_ARG_IS_EOL(args)) {
+		bool list = FALSE;
+
+		if (!str_append_nstring(str, args)) {
+			if (!imap_arg_get_list(args, &children)) {
+				/* everything is either nstring or list */
+				i_unreached();
+			}
+
+			str_append_c(str, '(');
+			imap_write_envelope_list(children, str, FALSE);
+			str_append_c(str, ')');
+
+			list = TRUE;
+		}
+		args++;
+
+		if ((toplevel || !list) && !IMAP_ARG_IS_EOL(args))
+			str_append_c(str, ' ');
+	}
+}
+
+static void
+imap_write_envelope(const struct imap_arg *args, string_t *str)
+{
+	imap_write_envelope_list(args, str, TRUE);
+}
+
 static int imap_parse_bodystructure_args(const struct imap_arg *args,
 					 string_t *str, const char **error_r)
 {