Cleanup

crates/store/re_chunk/src/chunk.rs

@@ -118,6 +118,7 @@ impl PartialEq for Chunk {
 impl Chunk {
     /// Returns a version of us with a new id
     #[must_use]
+    #[inline]
     pub fn with_id(mut self, id: ChunkId) -> Self {
         self.id = id;
         self

crates/viewer/re_ui/src/command.rs

@@ -298,7 +298,7 @@ impl UICommand {
             Self::CloseAllRecordings => None,
 
             Self::Undo => Some(cmd(Key::Z)),
-            Self::Redo => Some(cmd_shift(Key::Z)), // TODO: cmd-y?
+            Self::Redo => Some(cmd_shift(Key::Z)),
 
             #[cfg(all(not(target_arch = "wasm32"), target_os = "windows"))]
             Self::Quit => Some(KeyboardShortcut::new(Modifiers::ALT, Key::F4)),

crates/viewer/re_viewer/src/app_state.rs

@@ -504,7 +504,6 @@ impl AppState {
     /// If `inspect_blueprint_timeline` is enabled, we use the time selection from the
     /// blueprint `time_ctrl`. Otherwise, we use a latest query from the blueprint timeline.
     pub fn blueprint_query_for_viewer(&mut self, blueprint: &EntityDb) -> LatestAtQuery {
-        // TODO: sync self.blueprint_cfg.time_ctrl and blueprint_undo_state
         if self.app_options.inspect_blueprint_timeline {
             self.blueprint_cfg.time_ctrl.read().current_query().clone()
         } else {

crates/viewer/re_viewer/src/ui/rerun_menu.rs

@@ -32,8 +32,8 @@ impl App {
 
             ui.add_space(SPACING);
 
-            UICommand::Undo.menu_button_ui(ui, &self.command_sender); // TODO: only enabled if there is something to undo/redo
-            UICommand::Redo.menu_button_ui(ui, &self.command_sender); // TODO: only enabled if there is something to undo/redo
+            UICommand::Undo.menu_button_ui(ui, &self.command_sender); // TODO(emilk): only enabled if there is something to undo
+            UICommand::Redo.menu_button_ui(ui, &self.command_sender); // TODO(emilk): only enabled if there is something to redo
 
             UICommand::ToggleCommandPalette.menu_button_ui(ui, &self.command_sender);