summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJonas Dreßler <verdre@v0yd.nl>2022-10-30 10:30:42 +0100
committerMarge Bot <marge-bot@gnome.org>2022-11-04 18:47:27 +0000
commitc40feb33fab6bc258515a6b43a209a2f4bfbe981 (patch)
tree709892b07a640a6e08589dc2a2a8f469745136e9
parent0aba87308a99385145b05f1b18c194d7044bea5f (diff)
downloadmutter-c40feb33fab6bc258515a6b43a209a2f4bfbe981.tar.gz
clutter/actor: Remove outdated comment
Updating of the paint volume used for culling these days happens during the finish-layout stage, not while painting. Also we have geometry-based, not paint-based picking anymore. Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1492>
-rw-r--r--clutter/clutter/clutter-actor.c27
1 files changed, 0 insertions, 27 deletions
diff --git a/clutter/clutter/clutter-actor.c b/clutter/clutter/clutter-actor.c
index 0a53d9328..1dde8d948 100644
--- a/clutter/clutter/clutter-actor.c
+++ b/clutter/clutter/clutter-actor.c
@@ -3738,33 +3738,6 @@ clutter_actor_paint (ClutterActor *self,
*/
add_or_remove_flatten_effect (self);
- /* We save the current paint volume so that the next time the
- * actor queues a redraw we can constrain the redraw to just
- * cover the union of the new bounding box and the old.
- *
- * We also fetch the current paint volume to perform culling so
- * we can avoid painting actors outside the current clip region.
- *
- * If we are painting inside a clone, we should neither update
- * the paint volume or use it to cull painting, since the paint
- * box represents the location of the source actor on the
- * screen.
- *
- * XXX: We are starting to do a lot of vertex transforms on
- * the CPU in a typical paint, so at some point we should
- * audit these and consider caching some things.
- *
- * NB: We don't perform culling while picking at this point because
- * clutter-stage.c doesn't setup the clipping planes appropriately.
- *
- * NB: We don't want to update the last-paint-volume during picking
- * because the last-paint-volume is used to determine the old screen
- * space location of an actor that has moved so we can know the
- * minimal region to redraw to clear an old view of the actor. If we
- * update this during picking then by the time we come around to
- * paint then the last-paint-volume would likely represent the new
- * actor position not the old.
- */
culling_inhibited = priv->inhibit_culling_counter > 0;
if (!culling_inhibited && !in_clone_paint ())
{