GitLab

at 2024-08-12T12:37:23+02:00

widl: Explicitly check for top-level parameters before adding FC_ALLOCED_ON_STACK.

widl: Get rid of the write_embedded_types() helper.

It's barely doing anything anymore after
06e7eb5c98d473cf1edc828c67f6a6586a94e2f7.

widl: Ignore strings in is_embedded_complex().

Arrays declared as strings go via an embedded complex
(e.g. "[string] char s[64];").
This matches the logic used to decide struct or array FC.

This commit does nothing yet, since we pass NULL attributes, but we need to
change that so that other types are treated correctly (in particular,
void pointers with [iid_is].)

widl: Propagate attrs to inner pointer types.

Some attributes, namely [iid_is], [string], [switch_is], and [switch_type],
apply to the innermost type. That is, when specified on a top-level parameter
like

    int func(..., [iid_is(...)] void ****arg);

the innermost pointee ***arg must have an iid type.

At the same time, the pointer type attributes [ref], [unique], [ptr] must *only*
apply to the top-level pointer. Inner pointers have the default pointer type.

In order to implement this, we always pass "attrs" to inner array and pointer
types, and we also pass around an extra parameter "toplevel_attrs" which
signifies whether the passed-in attributes should be applied directly to this
type (having come from a parameter, field, or typedef), or whether they were
inherited from a higher-level pointer.

Alexandre Julliard pushed to branch master at wine / wine

Commits:

2 changed files: