safenames & sortcheck: add comments about safe unwraps

[skip ci]
dathere · Jan 21, 2023 · 59142b3 · 59142b3
1 parent f371ac2
commit 59142b3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/src/cmd/safenames.rs b/src/cmd/safenames.rs
@@ -237,6 +237,7 @@ pub fn run(argv: &[&str]) -> CliResult<()> {
                     unsafe_headers:  unsafenames_vec,
                     safe_headers:    safenames_vec,
                 };
+                // its OK to have unwrap here because safenames_struct is always valid
                 if safenames_mode == SafeNameMode::VerifyVerbosePrettyJSON {
                     println!(
                         "{}",

diff --git a/src/cmd/sortcheck.rs b/src/cmd/sortcheck.rs
@@ -209,6 +209,7 @@ pub fn run(argv: &[&str]) -> CliResult<()> {
             unsorted_breaks,
             dupe_count,
         };
+        // it's OK to have unwrap here as we know sortcheck_struct is valid json
         if args.flag_pretty_json {
             println!(
                 "{}",