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

Remove an extraneous setting of layer scope for vectors #60367

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

nyalldawson
Copy link
Collaborator

We already have set the layer scope for ALL layer types in the map renderer job preparation, so there's no need to add another layer scope in the vector layer renderer

Refs #60112

We already have set the layer scope for ALL layer types in
the map renderer job preparation, so there's no need to
add another layer scope in the vector layer renderer

Refs qgis#60112
@nyalldawson nyalldawson added the Optimization I feel the need... the need for speed! label Jan 31, 2025
@github-actions github-actions bot added this to the 3.42.0 milestone Jan 31, 2025
Copy link

github-actions bot commented Jan 31, 2025

🪟 Windows builds

Download Windows builds of this PR for testing.
Debug symbols for this build are available here.
(Built from commit 193f2de)

🪟 Windows Qt6 builds

Download Windows Qt6 builds of this PR for testing.
(Built from commit 193f2de)

@nyalldawson nyalldawson merged commit 5158569 into qgis:master Jan 31, 2025
39 checks passed
@nyalldawson nyalldawson deleted the remove_extraneous_layer_scope branch January 31, 2025 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Optimization I feel the need... the need for speed!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants