fix bug

Author: jfrolich

rewatch/src/build/clean.rs

@@ -170,18 +170,14 @@ pub fn cleanup_previous_build(
                 .get_mut(module_name)
                 .expect("Could not find module for ast file");
 
-            let compile_dirty = compile_assets_state.cmt_modules.get(module_name);
+            let cmt_last_modified = compile_assets_state.cmt_modules.get(module_name);
             // if there is a new AST but it has not been compiled yet, we mark the module as compile dirty
             // we do this by checking if the cmt file is newer than the AST file. We always compile the
             // interface AND implementation. For some reason the CMI file is not always rewritten if it
             // doesn't have any changes, that's why we just look at the CMT file.
-            if let Some(compile_dirty) = compile_dirty {
-                let last_modified = Some(ast_last_modified);
-
-                if let Some(last_modified) = last_modified {
-                    if compile_dirty > last_modified && !deleted_interfaces.contains(module_name) {
-                        module.compile_dirty = false;
-                    }
+            if let Some(cmt_last_modified) = cmt_last_modified {
+                if cmt_last_modified > ast_last_modified && !deleted_interfaces.contains(module_name) {
+                    module.compile_dirty = false;
                 }
             }
 

rewatch/src/helpers.rs

@@ -240,7 +240,9 @@ pub fn get_namespace_from_module_name(module_name: &str) -> Option<String> {
 }
 
 pub fn is_interface_ast_file(file: &Path) -> bool {
-    file.ends_with(".iast")
+    file.extension()
+        .map(|extension| extension.eq_ignore_ascii_case("iast"))
+        .unwrap_or(false)
 }
 
 pub fn read_lines(filename: &Path) -> io::Result<io::Lines<io::BufReader<fs::File>>> {
@@ -275,8 +277,7 @@ pub fn is_non_exotic_module_name(module_name: &str) -> bool {
 }
 
 pub fn get_extension(path: &Path) -> String {
-    let path_buf = PathBuf::from(path);
-    return path_buf
+    return path
         .extension()
         .expect("Could not get extension")
         .to_str()