Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ShellClients: Don't keep a clone if we never hide #1907

Merged
merged 2 commits into from
May 26, 2024

Conversation

leolost2605
Copy link
Member

@leolost2605 leolost2605 commented May 25, 2024

We don't need clone and hide tracker (which does quite a bunch of tracking) stuff if hide mode is set to never so don't create it in that case. Theoretically improves performance :)

@leolost2605 leolost2605 requested a review from a team May 25, 2024 10:20
@leolost2605 leolost2605 force-pushed the leolost/dont-track-if-not-needed branch from 23aae32 to e48cc5f Compare May 25, 2024 13:39
@leolost2605 leolost2605 changed the title ShellClients: Don't use a hide tracker if we never hide ShellClients: Don't keep a clone if we never hide May 25, 2024
@danirabbit danirabbit enabled auto-merge (squash) May 26, 2024 16:08
@danirabbit danirabbit merged commit fe650bf into master May 26, 2024
4 checks passed
@danirabbit danirabbit deleted the leolost/dont-track-if-not-needed branch May 26, 2024 16:08
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.

None yet

2 participants