ReceiveTempSMS
/
Sweden-Number
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

widl: Only base types or reference pointers to base types don't need to be freed. 

There are other cases where calls to Free functions can be omitted (such 
as types always unmarshalled using buffer memory with no embedded 
pointers), but these are easier dealt with inside the relevent case in 
write_remoting_arg.
This commit is contained in:
Rob Shearman 2007-12-15 13:58:17 +00:00 committed by Alexandre Julliard
parent 45d6a959fb
commit b81f4c24db
1 changed files with 3 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
tools/widl/typegen.c
View File

@ -2595,17 +2595,9 @@ static inline int is_size_needed_for_phase(enum remoting_phase phase)
return (phase != PHASE_UNMARSHAL); return (phase != PHASE_UNMARSHAL);
} }
static int needs_freeing(const attr_list_t *attrs, const type_t *t, int out) static int needs_freeing(const type_t *t)
{ {
return return !is_base_type(t->type) && (t->type != RPC_FC_RP || !is_base_type(t->ref->type));
(is_user_type(t)
|| (is_ptr(t)
&& (t->ref->type == RPC_FC_IP
|| (is_struct(t->ref->type) && t->ref->type != RPC_FC_STRUCT)
|| is_ptr(t->ref)
|| is_user_type(t->ref))))
|| (out && is_string_type(attrs, t))
|| is_array(t);
} }
expr_t *get_size_is_expr(const type_t *t, const char *name) expr_t *get_size_is_expr(const type_t *t, const char *name)
@ -2646,7 +2638,7 @@ static void write_remoting_arg(FILE *file, int indent, const func_t *func,
if (phase == PHASE_FREE) if (phase == PHASE_FREE)
{ {
if (!needs_freeing(var->attrs, type, out_attr)) if (!needs_freeing(type))
return; return;
} }
else else