Clean up some comments

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(),

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 {

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);
 

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>,
 }

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();
 

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`.