Page MenuHome
Differential D14129

Dual Mesh: use std::move
ClosedPublic
Actions

Authored by Wannes Malfait (Wannes) on Feb 16 2022, 4:32 PM.

Details

Reviewers
Hans Goudey (HooglyBoogly)
Commits
rBeeb0279e890e: Geometry Nodes: Use std::move in dual mesh node
Summary

Add a std::move in some places to prevent arrays from being copied.
These cases were most likely optimized by the compiler, but this makes
it more explicit.

Diff Detail

Repository
rB Blender

Event Timeline

Wannes Malfait (Wannes) requested review of this revision.Feb 16 2022, 4:32 PM
Wannes Malfait (Wannes) created this revision.
Harbormaster completed remote builds in B20600: Diff 48432.Feb 16 2022, 4:32 PM
Wannes Malfait (Wannes) added a reviewer: Hans Goudey (HooglyBoogly).Feb 16 2022, 4:33 PM
Wannes Malfait (Wannes) added a project: Geometry Nodes.
Hans Goudey (HooglyBoogly) accepted this revision.Feb 16 2022, 5:14 PM

Thanks :)

This revision is now accepted and ready to land.Feb 16 2022, 5:14 PM
Closed by commit rBeeb0279e890e: Geometry Nodes: Use std::move in dual mesh node (authored by Wannes Malfait (Wannes), committed by Hans Goudey (HooglyBoogly)). · Explain WhyFeb 28 2022, 9:16 PM
This revision was automatically updated to reflect the committed changes.
Hans Goudey (HooglyBoogly) added a commit: rBeeb0279e890e: Geometry Nodes: Use std::move in dual mesh node.

Revision Contents
Changeset List

PathSize
source/
blender/
nodes/
geometry/
nodes/
6 lines

Diff 48772

View Options

source/blender/nodes/geometry/nodes/node_geo_dual_mesh.cc

Loading...