review

Author: klensy

.github/workflows/spellcheck.yml

@@ -15,9 +15,9 @@ jobs:
         uses: actions/checkout@v4
 
       - name: check typos
-        # sync version with src/tools/tidy/src/ext_tool_checks.rs
+        # sync version with src/tools/tidy/src/ext_tool_checks.rs in spellcheck_runner
         uses: crate-ci/[email protected]
         with:
-          # sync target files with src/tools/tidy/src/ext_tool_checks.rs
+          # sync target files with src/tools/tidy/src/ext_tool_checks.rs in check_impl
           files: ./compiler ./library ./src/bootstrap ./src/librustdoc
           config: ./typos.toml

compiler/rustc_hir/src/hir.rs

@@ -4224,7 +4224,7 @@ impl fmt::Display for Constness {
     }
 }
 
-/// The actually safety specified in syntax. We may treat
+/// The actual safety specified in syntax. We may treat
 /// its safety different within the type system to create a
 /// "sound by default" system that needs checking this enum
 /// explicitly to allow unsafe operations.

library/panic_unwind/src/seh.rs

@@ -264,7 +264,7 @@ static mut TYPE_DESCRIPTOR: _TypeDescriptor = _TypeDescriptor {
 // runtime under a try/catch block and the panic that we generate here will be
 // used as the result of the exception copy. This is used by the C++ runtime to
 // support capturing exceptions with std::exception_ptr, which we can't support
-// because Box<dyn Any> isn't cloneable.
+// because Box<dyn Any> isn't clonable.
 macro_rules! define_cleanup {
     ($abi:tt $abi2:tt) => {
         unsafe extern $abi fn exception_cleanup(e: *mut Exception) {

src/tools/tidy/src/ext_tool_checks.rs

@@ -228,7 +228,7 @@ fn check_impl(
 
     if spellcheck_all || spellcheck_fix {
         let config_path = root_path.join("typos.toml");
-        // sync target files with .github/workflows/ci.yml
+        // sync target files with .github/workflows/spellcheck.yml
         let mut args = vec![
             "-c",
             config_path.as_os_str().to_str().unwrap(),
@@ -522,7 +522,7 @@ fn spellcheck_runner(args: &[&str]) -> Result<(), Error> {
             return Err(Error::MissingReq(
                 "typos",
                 "spellcheck file checks",
-                // sync version with .github/workflows/ci.yml
+                // sync version with .github/workflows/spellcheck.yml
                 Some("install tool via `cargo install [email protected]`".to_owned()),
             ));
         }

typos.toml

@@ -32,6 +32,7 @@ filetimes = "filetimes"
 misformed = "misformed"
 targetting = "targetting"
 publically = "publically"
+clonable = "clonable"
 
 # this can be valid word, depends on dictionary edition
 #matcheable = "matcheable"