Reformat

Author: lionel-

crates/oak_db/src/definition.rs

@@ -135,21 +135,13 @@ impl<'db> File {
 }
 
 /// Every `Definition` in a file, keyed by its definition site.
-///
-/// A `salsa::Update` struct rather than a bare map so the salsa-tracked
-/// `File::definitions` query can store it: salsa needs the `Update` impl to
-/// handle the `Definition<'db>` lifetime that the map values carry.
 #[derive(Debug, PartialEq, Eq, salsa::Update)]
 struct FileDefinitions<'db> {
     by_site: FxHashMap<DefinitionSite, Definition<'db>>,
 }
 
 /// Map key for [`FileDefinitions`]: a binding's `(scope, def_id)` site.
 ///
-/// A named struct rather than a `(ScopeId, DefinitionId)` tuple so it can
-/// derive `salsa::Update` (the salsa map key must implement it). The derive
-/// falls back to a plain replace for the `'static` index fields, which don't
-/// implement `Update` themselves (`oak_semantic` has no salsa dependency).
 /// Mirrors ty's `DefinitionNodeKey`.
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash, salsa::Update)]
 struct DefinitionSite {

crates/oak_db/src/tests/file_resolve.rs

@@ -388,7 +388,10 @@ fn test_same_name_sibling_insertion_churns_later_definition_id() {
     let file = files[0];
 
     // `resolve` is last-wins, so it returns the final `x` (`x <- 2`), ordinal 1.
-    let id1 = file.resolve(&db, name(&db, "x")).expect("x resolves").as_id();
+    let id1 = file
+        .resolve(&db, name(&db, "x"))
+        .expect("x resolves")
+        .as_id();
 
     // Insert another `x` at the top. The final `x` is still last-wins, but its
     // ordinal among same-name siblings shifts from 1 to 2.