djkato/egui
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Make it cheaper to clone Memory if there are a lot of persisted values

Browse source
This commit is contained in:
Emil Ernerfeldt 2021-10-27 16:49:08 +02:00
parent 3819ec6fc7
commit 081485bcaf
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
egui/src/util/id_type_map.rs
View file

@ -4,6 +4,7 @@
// This will also allow users to pick their own serialization format per type. // This will also allow users to pick their own serialization format per type.
use std::any::Any; use std::any::Any;
use std::sync::Arc;
// ----------------------------------------------------------------------------------------------- // -----------------------------------------------------------------------------------------------
@ -57,7 +58,7 @@ impl<T> SerializableAny for T where T: 'static + Any + Clone + for<'a> Send + Sy
#[cfg_attr(feature = "persistence", derive(serde::Deserialize, serde::Serialize))] #[cfg_attr(feature = "persistence", derive(serde::Deserialize, serde::Serialize))]
struct SerializedElement { struct SerializedElement {
type_id: TypeId, type_id: TypeId,
ron: String, ron: Arc<str>,
} }
#[cfg(feature = "persistence")] #[cfg(feature = "persistence")]
@ -82,7 +83,7 @@ enum Element {
/// The type of value we are storing. /// The type of value we are storing.
type_id: TypeId, type_id: TypeId,
/// The ron data we can deserialize. /// The ron data we can deserialize.
ron: String, ron: Arc<str>,
}, },
} }
@ -256,7 +257,7 @@ impl Element {
let ron = serialize_fn(value)?; let ron = serialize_fn(value)?;
Some(SerializedElement { Some(SerializedElement {
type_id: (**value).type_id().into(), type_id: (**value).type_id().into(),
ron, ron: ron.into(),
}) })
} else { } else {
None None