Skip to content

feat: Opt-out of LRU tracking by default statically#1051

Merged
Veykril merged 1 commit intosalsa-rs:masterfrom
Veykril:push-vvnoulxvzvlv
Jan 1, 2026
Merged

feat: Opt-out of LRU tracking by default statically#1051
Veykril merged 1 commit intosalsa-rs:masterfrom
Veykril:push-vvnoulxvzvlv

Conversation

@Veykril
Copy link
Member

@Veykril Veykril commented Dec 31, 2025

This gets rid of unnecessary branching in the majority of queries

@netlify
Copy link

netlify bot commented Dec 31, 2025

Deploy Preview for salsa-rs canceled.

Name Link
🔨 Latest commit 1e1e0c1
🔍 Latest deploy log https://app.netlify.com/projects/salsa-rs/deploys/6954eb946f54530008573a76

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 31, 2025

CodSpeed Performance Report

Merging #1051 will not alter performance

Comparing Veykril:push-vvnoulxvzvlv (1e1e0c1) with master (77c296f)

Summary

✅ 13 untouched

@Veykril Veykril changed the title feat: Allow opt-ing out of LRU at compile time feat: Opt-out of LRU tracking by default statically Dec 31, 2025
@Veykril Veykril marked this pull request as ready for review December 31, 2025 13:59
Copy link
Member

@ibraheemdev ibraheemdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@Veykril Veykril added this pull request to the merge queue Jan 1, 2026
Merged via the queue into salsa-rs:master with commit 23de837 Jan 1, 2026
14 checks passed
@Veykril Veykril deleted the push-vvnoulxvzvlv branch January 1, 2026 10:36
@github-actions github-actions bot mentioned this pull request Dec 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants