language-servers/slint
1
0
Fork 0
mirror of https://github.com/slint-ui/slint.git synced 2025-08-04 02:39:28 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Fix memory leak with repeated elements in generated Rust code

Browse source 
The generated code provides a PinnedDrop implementation that calls
free_component_item_graphics_resources. We must annotate the components
correctly to make sure to the drop implementation is called and graphics
resources are released.

Thanks to Jocelyn!

Fixes #1261
This commit is contained in:
Simon Hausmann 2022-06-08 15:54:30 +02:00
parent 7cead49f8c
commit f0c14bea54
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
internal/compiler/generator/rust.rs
View file

@ -121,7 +121,7 @@ pub fn generate(doc: &Document) -> TokenStream {
let sub_compos = llr
.sub_components
.iter()
.map(|sub_compo| generate_sub_component(sub_compo, &llr, None, quote!(), None))
.map(|sub_compo| generate_sub_component(sub_compo, &llr, None, quote!(), None, false))
.collect::<Vec<_>>();
let compo = generate_public_component(&llr);
@ -520,6 +520,7 @@ fn generate_sub_component(
parent_ctx: Option<ParentCtx>,
extra_fields: TokenStream,
index_property: Option<llr::PropertyIndex>,
pinned_drop: bool,
) -> TokenStream {
let inner_component_id = inner_component_id(component);
@ -776,11 +777,13 @@ fn generate_sub_component(
quote!(core::usize::MAX)
};
let pin_macro = if pinned_drop { quote!(#[pin_drop]) } else { quote!(#[pin]) };
quote!(
#[derive(slint::re_exports::FieldOffsets, Default)]
#[const_field_offset(slint::re_exports::const_field_offset)]
#[repr(C)]
#[pin]
#pin_macro
#visibility
struct #inner_component_id {
#(#item_names : slint::re_exports::#item_types,)*
@ -998,6 +1001,7 @@ fn generate_item_tree(
parent_ctx.clone(),
extra_fields,
index_property,
true,
);
let inner_component_id = self::inner_component_id(&sub_tree.root);
let parent_component_type = parent_ctx.iter().map(|parent| {