eliminate leftover events system, since ui codepaths should now consume as many events as possible within a single frame

This commit is contained in:
Ryan Fleury
2024-02-08 10:09:51 -08:00
parent 6e3242848f
commit fed0a13aca
2 changed files with 1 additions and 29 deletions
+1 -27
View File
@@ -65,10 +65,7 @@ update_and_render(OS_Handle repaint_window_handle, void *user_data)
OS_EventList events = {0};
if(os_handle_match(repaint_window_handle, os_handle_zero()))
{
OS_EventList leftover_events_copy = os_event_list_copy(scratch.arena, &leftover_events);
OS_EventList new_events = os_get_events(scratch.arena, df_gfx_state->num_frames_requested == 0);
os_event_list_concat_in_place(&events, &leftover_events_copy);
os_event_list_concat_in_place(&events, &new_events);
events = os_get_events(scratch.arena, df_gfx_state->num_frames_requested == 0);
}
//- rjf: enable txti change detection
@@ -321,26 +318,6 @@ update_and_render(OS_Handle repaint_window_handle, void *user_data)
}
}
//- rjf: gather leftover events for subsequent frame
// TODO(rjf): need to prevent spurious unconsumed events in UI layer & other event handling paths
if(events.count != 0 && 0)
{
arena_clear(leftover_events_arena);
leftover_events = os_event_list_copy(leftover_events_arena, &events);
for(OS_Event *ev = leftover_events.first, *next = 0; ev != 0; ev = next)
{
next = ev->next;
if(ev->timestamp_us+1000000 < os_now_microseconds() ||
ev->kind == OS_EventKind_Text ||
(ev->kind == OS_EventKind_Press && ev->key != OS_Key_LeftMouseButton && ev->key != OS_Key_RightMouseButton && ev->key != OS_Key_MiddleMouseButton) ||
(ev->kind == OS_EventKind_Release && ev->key != OS_Key_LeftMouseButton && ev->key != OS_Key_RightMouseButton && ev->key != OS_Key_MiddleMouseButton) ||
(ev->kind == OS_EventKind_Scroll))
{
os_eat_event(&leftover_events, ev);
}
}
}
//- rjf: determine frame time, record into history
U64 end_time_us = os_now_microseconds();
U64 frame_time_us = end_time_us-begin_time_us;
@@ -449,9 +426,6 @@ entry_point(int argc, char **argv)
IPCInfo *ipc_info = (IPCInfo *)ipc_shared_memory_base;
ipc_info->msg_size = 0;
//- rjf: set up leftover event arena
leftover_events_arena = arena_alloc();
//- rjf: initialize stuff we depend on
{
hs_init();
-2
View File
@@ -446,8 +446,6 @@ read_only global String8 ipc_shared_memory_name = str8_lit_comp("_raddbg_ipc_sha
read_only global String8 ipc_semaphore_name = str8_lit_comp("_raddbg_ipc_semaphore_");
global U64 frame_time_us_history[64] = {0};
global U64 frame_time_us_history_idx = 0;
global Arena *leftover_events_arena = 0;
global OS_EventList leftover_events = {0};
////////////////////////////////
//~ rjf: Frontend Entry Points