Skip to content

Commit

Permalink
Merge pull request #736 from Veykril/veykril/push-mntwomxrrovr
Browse files Browse the repository at this point in the history
Remove unnecessary `current_revision` call from `setup_interned_struct`
  • Loading branch information
Veykril authored Feb 26, 2025
2 parents 2fc72ba + 870dbfe commit 26aeeec
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion components/salsa-macro-rules/src/setup_interned_struct.rs
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,6 @@ macro_rules! setup_interned_struct {
$field_ty: $zalsa::interned::HashEqLike<$indexed_ty>,
)*
{
let current_revision = $zalsa::current_revision(db);
$Configuration::ingredient(db).intern(db.as_dyn_database(),
StructKey::<$db_lt>($($field_id,)* std::marker::PhantomData::default()), |_, data| ($($zalsa::interned::Lookup::into_owned(data.$field_index),)*))
}
Expand Down
1 change: 0 additions & 1 deletion tests/interned-structs_self_ref.rs
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,6 @@ const _: () = {
Db_: ?Sized + salsa::Database,
String: zalsa_::interned::HashEqLike<T0>,
{
let current_revision = zalsa_::current_revision(db);
Configuration_::ingredient(db).intern(
db.as_dyn_database(),
StructKey::<'db>(data, std::marker::PhantomData::default()),
Expand Down

0 comments on commit 26aeeec

Please sign in to comment.