Merge pull request #2 from JMS55/24

Don't support diagnostic filters for now

src/compose/mod.rs

@@ -1045,7 +1045,7 @@ impl Composer {
                         expressions: Default::default(),
                         named_expressions: Default::default(),
                         body: Default::default(),
-                        diagnostic_filter_leaf: todo!(),
+                        diagnostic_filter_leaf: None,
                     };
 
                     // record owned function
@@ -1083,7 +1083,7 @@ impl Composer {
                     expressions: Default::default(),
                     named_expressions: Default::default(),
                     body: Default::default(),
-                    diagnostic_filter_leaf: todo!(),
+                    diagnostic_filter_leaf: None,
                 };
 
                 owned_functions.insert(ep.function.name.clone().unwrap(), (None, header_function));

src/compose/test.rs

@@ -1264,6 +1264,7 @@ mod test {
     }
 
     #[test]
+    #[should_panic] // Diagnostic filters not yet supported
     fn test_diagnostic_filters() {
         let mut composer = Composer::default();
 

src/derive.rs

@@ -792,7 +792,7 @@ impl<'a> DerivedModule<'a> {
             expressions: Rc::try_unwrap(expressions).unwrap().into_inner(),
             named_expressions,
             body,
-            diagnostic_filter_leaf: todo!(),
+            diagnostic_filter_leaf: None,
         }
     }
 
@@ -867,8 +867,8 @@ impl From<DerivedModule<'_>> for naga::Module {
             special_types: Default::default(),
             entry_points: Default::default(),
             overrides: derived.pipeline_overrides,
-            diagnostic_filters: todo!(),
-            diagnostic_filter_leaf: todo!(),
+            diagnostic_filters: Default::default(),
+            diagnostic_filter_leaf: None,
         }
     }
 }