Clean up some comments

mtkennerly · Nov 18, 2024 · cdb2c70 · cdb2c70
1 parent 93a14c1
commit cdb2c70
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 7 deletions.
diff --git a/src/gui/popup_menu.rs b/src/gui/popup_menu.rs
@@ -223,7 +223,6 @@ impl<T> State<T> {
     pub fn new() -> Self {
         Self {
             menu: menu::State::default(),
-            // keyboard_modifiers: keyboard::Modifiers::default(),
             is_open: bool::default(),
             hovered_option: Option::default(),
             last_selection: Option::default(),

diff --git a/src/lang.rs b/src/lang.rs
@@ -46,7 +46,7 @@ fn title_case(text: &str) -> String {
     }
 }
 
-// TODO: Some are blocked by https://github.com/mtkennerly/ludusavi/issues/9.
+// TODO: RTL blocked by https://github.com/iced-rs/iced/issues/250.
 /// Display language.
 #[derive(Clone, Copy, Debug, Default, Eq, PartialEq, serde::Serialize, serde::Deserialize, schemars::JsonSchema)]
 pub enum Language {

diff --git a/src/prelude.rs b/src/prelude.rs
@@ -39,7 +39,6 @@ pub static OS_USERNAME: Lazy<String> = Lazy::new(whoami::username);
 
 pub static AVAILABLE_PARALELLISM: Lazy<Option<NonZeroUsize>> = Lazy::new(|| std::thread::available_parallelism().ok());
 
-// NOTE.2022-11-04 not very pretty singleton like global variable
 pub static CONFIG_DIR: Mutex<Option<PathBuf>> = Mutex::new(None);
 static HANDLER_SIGINT: Mutex<Option<signal_hook::SigId>> = Mutex::new(None);
 

diff --git a/src/scan/backup.rs b/src/scan/backup.rs
@@ -27,7 +27,6 @@ impl BackupError {
 
 #[derive(Clone, Debug, Default)]
 pub struct BackupInfo {
-    // TODO: Use `StrictPath` as key instead of whole `ScannedFile`?
     pub failed_files: HashMap<StrictPath, BackupError>,
     pub failed_registry: HashMap<RegistryItem, BackupError>,
 }

diff --git a/src/scan/launchers/heroic/gog.rs b/src/scan/launchers/heroic/gog.rs
@@ -58,7 +58,6 @@ pub fn scan(root: &root::Heroic, title_finder: &TitleFinder) -> HashMap<String,
         return games;
     }
 
-    // iterate over all games found in HEROCONFIGDIR/gog_store/installed.json and call find_prefix
     let installed_path = root.path.joined(installed::PATH);
     let content = installed_path.read();
 

diff --git a/src/scan/title.rs b/src/scan/title.rs
@@ -253,13 +253,13 @@ pub struct TitleQuery {
     /// Search for exact titles or aliases.
     /// This will cause only one result to be returned.
     pub names: Vec<String>,
-    // Search for a Steam ID.
+    /// Search for a Steam ID.
     /// This will cause only one result to be returned.
     pub steam_id: Option<u32>,
     /// Search for a GOG ID.
     /// This will cause only one result to be returned.
     pub gog_id: Option<u64>,
-    // Search for a Lutris slug.
+    /// Search for a Lutris slug.
     /// This will cause only one result to be returned.
     pub lutris_id: Option<String>,
     /// Search by normalizing the `names`.