Reduce some excessive indentation.

I prefer checking a condition and returning early rather than making
large blocks of code within a function conditional.
This commit is contained in:
Carl Worth 2011-05-31 16:18:45 -07:00
parent f744b050b1
commit 2d1356e8db

View file

@ -393,22 +393,23 @@ show_text_part_content (GMimeObject *part, GMimeStream *stream_out)
INTERNAL_ERROR ("Illegal request to format non-text part (%s) as text.", INTERNAL_ERROR ("Illegal request to format non-text part (%s) as text.",
g_mime_content_type_to_string (content_type)); g_mime_content_type_to_string (content_type));
charset = g_mime_object_get_content_type_parameter (part, "charset"); if (stream_out == NULL)
return;
if (stream_out) { stream_filter = g_mime_stream_filter_new (stream_out);
stream_filter = g_mime_stream_filter_new (stream_out); g_mime_stream_filter_add(GMIME_STREAM_FILTER (stream_filter),
g_mime_stream_filter_add(GMIME_STREAM_FILTER (stream_filter), g_mime_filter_crlf_new (FALSE, FALSE));
g_mime_filter_crlf_new (FALSE, FALSE));
if (charset) { charset = g_mime_object_get_content_type_parameter (part, "charset");
GMimeFilter *charset_filter; if (charset) {
charset_filter = g_mime_filter_charset_new (charset, "UTF-8"); GMimeFilter *charset_filter;
/* This result can be NULL for things like "unknown-8bit". charset_filter = g_mime_filter_charset_new (charset, "UTF-8");
* Don't set a NULL filter as that makes GMime print /* This result can be NULL for things like "unknown-8bit".
* annoying assertion-failure messages on stderr. */ * Don't set a NULL filter as that makes GMime print
if (charset_filter) * annoying assertion-failure messages on stderr. */
g_mime_stream_filter_add (GMIME_STREAM_FILTER (stream_filter), if (charset_filter)
charset_filter); g_mime_stream_filter_add (GMIME_STREAM_FILTER (stream_filter),
} charset_filter);
} }
wrapper = g_mime_part_get_content_object (GMIME_PART (part)); wrapper = g_mime_part_get_content_object (GMIME_PART (part));