mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2026-06-05 23:48:17 +02:00
b824ef83ab
Most of the time, we can infer the type to append in
util_dynarray_append using __typeof__, which is standardized in C23 and
support in Jesse's MSMSVCV. This patch drops the type argument most of
the time, making util_dynarray a little more ergonomic to use.
This is done in four steps.
First, rename util_dynarray_append -> util_dynarray_append_typed
bash -c "find . -type f -exec sed -i -e 's/util_dynarray_append(/util_dynarray_append_typed(/g' \{} \;"
Then, add a new append that infers the type. This is much more ergonomic
for what you want most of the time.
Next, use type-inferred append as much as possible, via Coccinelle
patch (plus manual fixup):
@@
expression dynarray, element;
type type;
@@
-util_dynarray_append_typed(dynarray, type, element);
+util_dynarray_append(dynarray, element);
Finally, hand fixup cases that Coccinelle missed or incorrectly
translated, of which there were several because we can't used the
untyped append with a literal (since the sizeof won't do what you want).
All four steps are squashed to produce a single patch changing every
util_dynarray_append call site in tree to either drop a type parameter
(if possible) or insert a _typed suffix (if we can't infer). As such,
the final patch is best reviewed by hand even though it was
tool-assisted.
No Long Linguine Meals were involved in the making of this patch.
Signed-off-by: Alyssa Rosenzweig <alyssa.rosenzweig@intel.com>
Acked-by: Faith Ekstrand <faith.ekstrand@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/38038>
|
||
|---|---|---|
| .. | ||
| tests | ||
| agx_abi.h | ||
| agx_bg_eot.c | ||
| agx_bg_eot.h | ||
| agx_bo.c | ||
| agx_bo.h | ||
| agx_border.c | ||
| agx_device.c | ||
| agx_device.h | ||
| agx_device_virtio.c | ||
| agx_device_virtio.h | ||
| agx_helpers.h | ||
| agx_linker.c | ||
| agx_linker.h | ||
| agx_nir_format_helpers.h | ||
| agx_nir_lower_msaa.c | ||
| agx_nir_lower_sample_intrinsics.c | ||
| agx_nir_lower_tilebuffer.c | ||
| agx_nir_lower_uvs.c | ||
| agx_nir_lower_vbo.c | ||
| agx_nir_lower_vbo.h | ||
| agx_nir_prolog_epilog.c | ||
| agx_ppp.h | ||
| agx_scratch.c | ||
| agx_scratch.h | ||
| agx_tilebuffer.c | ||
| agx_tilebuffer.h | ||
| agx_usc.h | ||
| agx_uvs.h | ||
| agx_va.c | ||
| asahi_proto.h | ||
| decode.c | ||
| decode.h | ||
| dyld_interpose.h | ||
| meson.build | ||
| pool.c | ||
| pool.h | ||
| wrap.c | ||