From 1be8dd71c8d35d4260e66ae3c5a75169c6b159bd Mon Sep 17 00:00:00 2001
From: Jeremy Leibs <jeremy@rerun.io>
Date: Wed, 12 Apr 2023 16:09:02 +0200
Subject: [PATCH] PR tweaks

---
 crates/re_viewer/src/web.rs | 7 +++++++
 crates/rerun/src/run.rs     | 7 ++++++-
 2 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/crates/re_viewer/src/web.rs b/crates/re_viewer/src/web.rs
index 85e1e8103a93..2f214e1073a4 100644
--- a/crates/re_viewer/src/web.rs
+++ b/crates/re_viewer/src/web.rs
@@ -134,6 +134,13 @@ fn get_persist_state(info: &eframe::IntegrationInfo) -> bool {
         .map(String::as_str)
     {
         Some("0") => false,
+        Some("1") => true,
+        Some(other) => {
+            re_log::warn!(
+                "Unexpected value for 'persist' query: {other:?}. Expected either '0' or '1'. Defaulting to '1'."
+            );
+            true
+        }
         _ => true,
     }
 }
diff --git a/crates/rerun/src/run.rs b/crates/rerun/src/run.rs
index 9348b8a1a492..8e05289a61af 100644
--- a/crates/rerun/src/run.rs
+++ b/crates/rerun/src/run.rs
@@ -58,7 +58,12 @@ struct Args {
     #[clap(long)]
     memory_limit: Option<String>,
 
-    /// Whether the Rerun Viewer should persist the state of the viewer to local storage
+    /// Whether the Rerun Viewer should persist the state of the viewer to disk.
+    ///
+    /// When persisted, the state will be stored at the following locations:
+    /// - Linux: /home/UserName/.local/share/rerunviewer
+    /// - macOS: /Users/UserName/Library/Application Support/rerunviewer
+    /// - Windows: C:\Users\UserName\AppData\Roaming\rerunviewer
     #[clap(long, default_value_t = true)]
     persist_state: bool,