remove boolean "first" argument from format_part_sprinter

This argument seems to be unused, and format_part_sprinter isn't
required to meet any specific API so it seems cleaner and simpler to
drop it.
This commit is contained in:
Daniel Kahn Gillmor 2017-07-14 15:05:06 +02:00 committed by David Bremner
parent f385055528
commit 23eed88e32
3 changed files with 6 additions and 6 deletions

View file

@ -247,7 +247,7 @@ show_one_part (const char *filename, int part);
void void
format_part_sprinter (const void *ctx, struct sprinter *sp, mime_node_t *node, format_part_sprinter (const void *ctx, struct sprinter *sp, mime_node_t *node,
notmuch_bool_t first, notmuch_bool_t output_body, notmuch_bool_t output_body,
notmuch_bool_t include_html); notmuch_bool_t include_html);
void void

View file

@ -667,7 +667,7 @@ static int do_reply(notmuch_config_t *config,
/* Start the original */ /* Start the original */
sp->map_key (sp, "original"); sp->map_key (sp, "original");
format_part_sprinter (config, sp, node, TRUE, TRUE, FALSE); format_part_sprinter (config, sp, node, TRUE, FALSE);
/* End */ /* End */
sp->end (sp); sp->end (sp);

View file

@ -603,7 +603,7 @@ format_omitted_part_meta_sprinter (sprinter_t *sp, GMimeObject *meta, GMimePart
void void
format_part_sprinter (const void *ctx, sprinter_t *sp, mime_node_t *node, format_part_sprinter (const void *ctx, sprinter_t *sp, mime_node_t *node,
notmuch_bool_t first, notmuch_bool_t output_body, notmuch_bool_t output_body,
notmuch_bool_t include_html) notmuch_bool_t include_html)
{ {
/* Any changes to the JSON or S-Expression format should be /* Any changes to the JSON or S-Expression format should be
@ -619,7 +619,7 @@ format_part_sprinter (const void *ctx, sprinter_t *sp, mime_node_t *node,
if (output_body) { if (output_body) {
sp->map_key (sp, "body"); sp->map_key (sp, "body");
sp->begin_list (sp); sp->begin_list (sp);
format_part_sprinter (ctx, sp, mime_node_child (node, 0), first, TRUE, include_html); format_part_sprinter (ctx, sp, mime_node_child (node, 0), TRUE, include_html);
sp->end (sp); sp->end (sp);
} }
sp->end (sp); sp->end (sp);
@ -721,7 +721,7 @@ format_part_sprinter (const void *ctx, sprinter_t *sp, mime_node_t *node,
} }
for (i = 0; i < node->nchildren; i++) for (i = 0; i < node->nchildren; i++)
format_part_sprinter (ctx, sp, mime_node_child (node, i), i == 0, TRUE, include_html); format_part_sprinter (ctx, sp, mime_node_child (node, i), TRUE, include_html);
/* Close content structures */ /* Close content structures */
for (i = 0; i < nclose; i++) for (i = 0; i < nclose; i++)
@ -735,7 +735,7 @@ format_part_sprinter_entry (const void *ctx, sprinter_t *sp,
mime_node_t *node, unused (int indent), mime_node_t *node, unused (int indent),
const notmuch_show_params_t *params) const notmuch_show_params_t *params)
{ {
format_part_sprinter (ctx, sp, node, TRUE, params->output_body, params->include_html); format_part_sprinter (ctx, sp, node, params->output_body, params->include_html);
return NOTMUCH_STATUS_SUCCESS; return NOTMUCH_STATUS_SUCCESS;
} }