From dd58fcc10edb1d3b9d96151cded067e1631a54ed Mon Sep 17 00:00:00 2001 From: Jacek Caban Date: Sun, 26 Jul 2015 18:07:37 +0200 Subject: [PATCH] widl: Don't output C++ default value if the argument is followed by another without default value. --- tools/widl/header.c | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/tools/widl/header.c b/tools/widl/header.c index c23faf49f96..bf824d8bc3e 100644 --- a/tools/widl/header.c +++ b/tools/widl/header.c @@ -937,8 +937,18 @@ void write_args(FILE *h, const var_list_t *args, const char *name, int method, i if (method == 2) { const expr_t *expr = get_attrp(arg->attrs, ATTR_DEFAULTVALUE); if (expr) { - fprintf(h, " = "); - write_expr( h, expr, 0, 1, NULL, NULL, "" ); + const var_t *tail_arg; + + /* Output default value only if all following arguments also have default value. */ + LIST_FOR_EACH_ENTRY_REV( tail_arg, args, const var_t, entry ) { + if(tail_arg == arg) { + fprintf(h, " = "); + write_expr( h, expr, 0, 1, NULL, NULL, "" ); + break; + } + if(!get_attrp(tail_arg->attrs, ATTR_DEFAULTVALUE)) + break; + } } } count++; -- 2.11.4.GIT