From: Hans Leidekker hans@codeweavers.com
--- tools/widl/parser.y | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+)
diff --git a/tools/widl/parser.y b/tools/widl/parser.y index a9bb80f9212..e96e57ad360 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -2647,6 +2647,24 @@ static void check_propget_args( const var_t *func, const var_list_t *args ) } }
+static void check_propput_args( const var_t *func, const var_list_t *args ) +{ + const var_t *arg; + unsigned int count = 0; + + LIST_FOR_EACH_ENTRY_REV( arg, args, const var_t, entry ) + { + int is_size = is_size_parameter( arg, args ); + + count++; + if (is_attr( arg->attrs, ATTR_OUT )) + error_at( &arg->where, "parameter '%s' of function '%s' must be an [in] parameter\n", arg->name, func->name ); + + if ((is_size && count > 2) || (!is_size && count > 1)) + error_at( &arg->where, "propput function '%s' has too many parameters\n", func->name ); + } +} + /* checks that arguments for a function make sense for marshalling and unmarshalling */ static void check_remoting_args(const var_t *func) { @@ -2782,6 +2800,7 @@ static void check_functions(const type_t *iface, int is_inside_library) if (args && is_attr( func->attrs, ATTR_EVENTADD )) check_eventadd_args( func, args ); if (args && is_attr( func->attrs, ATTR_EVENTREMOVE )) check_eventremove_args( func, args ); if (args && is_attr( func->attrs, ATTR_PROPGET )) check_propget_args( func, args ); + if (args && is_attr( func->attrs, ATTR_PROPPUT )) check_propput_args( func, args ); } } }