diff --git a/crates/bevy_app/src/app.rs b/crates/bevy_app/src/app.rs index b13d9d7700b41d..fa00654ed265d4 100644 --- a/crates/bevy_app/src/app.rs +++ b/crates/bevy_app/src/app.rs @@ -213,6 +213,7 @@ pub enum PluginsState { // Dummy plugin used to temporary hold the place in the plugin registry struct PlaceholderPlugin; + impl Plugin for PlaceholderPlugin { fn build(&self, _app: &mut App) {} } @@ -505,6 +506,7 @@ impl App { self.init_resource::>().add_systems( First, bevy_ecs::event::event_update_system:: + .in_set(bevy_ecs::event::EventUpdates) .run_if(bevy_ecs::event::event_update_condition::), ); } diff --git a/crates/bevy_ecs/src/event.rs b/crates/bevy_ecs/src/event.rs index 554b17195fbeb5..62cfce6329a61f 100644 --- a/crates/bevy_ecs/src/event.rs +++ b/crates/bevy_ecs/src/event.rs @@ -3,6 +3,7 @@ use crate as bevy_ecs; use crate::system::{Local, Res, ResMut, Resource, SystemParam}; pub use bevy_ecs_macros::Event; +use bevy_ecs_macros::SystemSet; use bevy_utils::detailed_trace; use std::ops::{Deref, DerefMut}; use std::{ @@ -13,6 +14,7 @@ use std::{ marker::PhantomData, slice::Iter, }; + /// A type that can be stored in an [`Events`] resource /// You can conveniently access events using the [`EventReader`] and [`EventWriter`] system parameter. /// @@ -33,6 +35,7 @@ pub struct EventId { } impl Copy for EventId {} + impl Clone for EventId { fn clone(&self) -> Self { *self @@ -790,22 +793,33 @@ impl<'a, E: Event> ExactSizeIterator for EventIteratorWithId<'a, E> { #[derive(Resource, Default)] pub struct EventUpdateSignal(bool); -/// A system that queues a call to [`Events::update`]. -pub fn event_queue_update_system(signal: Option>) { +#[doc(hidden)] +#[derive(SystemSet, Clone, Debug, PartialEq, Eq, Hash)] +pub struct EventUpdates; + +/// Signals the [`event_update_system`] to run after `FixedUpdate` systems. +pub fn signal_event_update_system(signal: Option>) { if let Some(mut s) = signal { s.0 = true; } } +/// Resets the `EventUpdateSignal` +pub fn reset_event_update_signal_system(signal: Option>) { + if let Some(mut s) = signal { + s.0 = false; + } +} + /// A system that calls [`Events::update`]. pub fn event_update_system( - signal: Option>, + update_signal: Option>, mut events: ResMut>, ) { - if let Some(mut s) = signal { + if let Some(signal) = update_signal { // If we haven't got a signal to update the events, but we *could* get such a signal // return early and update the events later. - if !std::mem::replace(&mut s.0, false) { + if !signal.0 { return; } } diff --git a/crates/bevy_time/src/lib.rs b/crates/bevy_time/src/lib.rs index 9b7e7a1ea21cc5..fd2134e0f031b7 100644 --- a/crates/bevy_time/src/lib.rs +++ b/crates/bevy_time/src/lib.rs @@ -25,7 +25,7 @@ pub mod prelude { } use bevy_app::{prelude::*, RunFixedMainLoop}; -use bevy_ecs::event::{event_queue_update_system, EventUpdateSignal}; +use bevy_ecs::event::{signal_event_update_system, EventUpdateSignal, EventUpdates}; use bevy_ecs::prelude::*; use bevy_utils::{tracing::warn, Duration, Instant}; pub use crossbeam_channel::TrySendError; @@ -61,7 +61,11 @@ impl Plugin for TimePlugin { // ensure the events are not dropped until `FixedMain` systems can observe them app.init_resource::() - .add_systems(FixedPostUpdate, event_queue_update_system); + .add_systems( + First, + bevy_ecs::event::reset_event_update_signal_system.after(EventUpdates), + ) + .add_systems(FixedPostUpdate, signal_event_update_system); #[cfg(feature = "bevy_ci_testing")] if let Some(ci_testing_config) = app @@ -142,3 +146,65 @@ fn time_system( TimeUpdateStrategy::ManualDuration(duration) => time.update_with_duration(*duration), } } + +#[cfg(test)] +mod tests { + use crate::{Fixed, Time, TimePlugin, TimeUpdateStrategy}; + use bevy_app::{App, Startup, Update}; + use bevy_ecs::event::{Event, EventReader, EventWriter}; + use std::error::Error; + + #[derive(Event)] + struct TestEvent { + sender: std::sync::mpsc::Sender, + } + + impl Drop for TestEvent { + fn drop(&mut self) { + self.sender + .send(T::default()) + .expect("Failed to send drop signal"); + } + } + + #[test] + fn events_get_dropped_regression_test_11528() -> Result<(), impl Error> { + let (tx1, rx1) = std::sync::mpsc::channel(); + let (tx2, rx2) = std::sync::mpsc::channel(); + let mut app = App::new(); + app.add_plugins(TimePlugin) + .add_event::>() + .add_event::>() + .add_systems(Startup, move |mut ev2: EventWriter>| { + ev2.send(TestEvent { + sender: tx2.clone(), + }); + }) + .add_systems(Update, move |mut ev1: EventWriter>| { + // Keep adding events so this event type is processed every update + ev1.send(TestEvent { + sender: tx1.clone(), + }); + }) + .add_systems( + Update, + |mut ev1: EventReader>, mut ev2: EventReader>| { + // Read events so they can be dropped + for _ in ev1.read() {} + for _ in ev2.read() {} + }, + ) + .insert_resource(TimeUpdateStrategy::ManualDuration( + Time::::default().timestep(), + )); + + for _ in 0..10 { + app.update(); + } + + // Check event type 1 as been dropped at least once + let _drop_signal = rx1.try_recv()?; + // Check event type 2 has been dropped + rx2.try_recv() + } +}