Cleanup: Complete earlier geometry component refactor

This was meant to be part of rB9ce950daabbf, but the change dropped from
the set at some point in the process of updating and committing.
Sorry for the noise.

1 files changed, 0 insertions, 431 deletions

diff --git a/source/blender/blenkernel/intern/geometry_set.cc b/source/blender/blenkernel/intern/geometry_set.cc
index 0274dfdbd1c..e7fb184023d 100644
--- a/source/blender/blenkernel/intern/geometry_set.cc
+++ b/source/blender/blenkernel/intern/geometry_set.cc
@@ -311,437 +311,6 @@ Volume *GeometrySet::get_volume_for_write()
 /** \} */
 
 /* -------------------------------------------------------------------- */
-/** \name Mesh Component
- * \{ */
-
-MeshComponent::MeshComponent() : GeometryComponent(GeometryComponentType::Mesh)
-{
-}
-
-MeshComponent::~MeshComponent()
-{
-  this->clear();
-}
-
-GeometryComponent *MeshComponent::copy() const
-{
-  MeshComponent *new_component = new MeshComponent();
-  if (mesh_ != nullptr) {
-    new_component->mesh_ = BKE_mesh_copy_for_eval(mesh_, false);
-    new_component->ownership_ = GeometryOwnershipType::Owned;
-    new_component->vertex_group_names_ = blender::Map(vertex_group_names_);
-  }
-  return new_component;
-}
-
-void MeshComponent::clear()
-{
-  BLI_assert(this->is_mutable());
-  if (mesh_ != nullptr) {
-    if (ownership_ == GeometryOwnershipType::Owned) {
-      BKE_id_free(nullptr, mesh_);
-    }
-    mesh_ = nullptr;
-  }
-  vertex_group_names_.clear();
-}
-
-bool MeshComponent::has_mesh() const
-{
-  return mesh_ != nullptr;
-}
-
-/* Clear the component and replace it with the new mesh. */
-void MeshComponent::replace(Mesh *mesh, GeometryOwnershipType ownership)
-{
-  BLI_assert(this->is_mutable());
-  this->clear();
-  mesh_ = mesh;
-  ownership_ = ownership;
-}
-
-/* This function exists for the same reason as #vertex_group_names_. Non-nodes modifiers need to
- * be able to replace the mesh data without losing the vertex group names, which may have come
- * from another object. */
-void MeshComponent::replace_mesh_but_keep_vertex_group_names(Mesh *mesh,
-                                                             GeometryOwnershipType ownership)
-{
-  BLI_assert(this->is_mutable());
-  if (mesh_ != nullptr) {
-    if (ownership_ == GeometryOwnershipType::Owned) {
-      BKE_id_free(nullptr, mesh_);
-    }
-    mesh_ = nullptr;
-  }
-  mesh_ = mesh;
-  ownership_ = ownership;
-}
-
-/* Return the mesh and clear the component. The caller takes over responsibility for freeing the
- * mesh (if the component was responsible before). */
-Mesh *MeshComponent::release()
-{
-  BLI_assert(this->is_mutable());
-  Mesh *mesh = mesh_;
-  mesh_ = nullptr;
-  return mesh;
-}
-
-void MeshComponent::copy_vertex_group_names_from_object(const Object &object)
-{
-  BLI_assert(this->is_mutable());
-  vertex_group_names_.clear();
-  int index = 0;
-  LISTBASE_FOREACH (const bDeformGroup *, group, &object.defbase) {
-    vertex_group_names_.add(group->name, index);
-    index++;
-  }
-}
-
-const blender::Map<std::string, int> &MeshComponent::vertex_group_names() const
-{
-  return vertex_group_names_;
-}
-
-/* This is only exposed for the internal attribute API. */
-blender::Map<std::string, int> &MeshComponent::vertex_group_names()
-{
-  return vertex_group_names_;
-}
-
-/* Get the mesh from this component. This method can be used by multiple threads at the same
- * time. Therefore, the returned mesh should not be modified. No ownership is transferred. */
-const Mesh *MeshComponent::get_for_read() const
-{
-  return mesh_;
-}
-
-/* Get the mesh from this component. This method can only be used when the component is mutable,
- * i.e. it is not shared. The returned mesh can be modified. No ownership is transferred. */
-Mesh *MeshComponent::get_for_write()
-{
-  BLI_assert(this->is_mutable());
-  if (ownership_ == GeometryOwnershipType::ReadOnly) {
-    mesh_ = BKE_mesh_copy_for_eval(mesh_, false);
-    ownership_ = GeometryOwnershipType::Owned;
-  }
-  return mesh_;
-}
-
-bool MeshComponent::is_empty() const
-{
-  return mesh_ == nullptr;
-}
-
-/** \} */
-
-/* -------------------------------------------------------------------- */
-/** \name Pointcloud Component
- * \{ */
-
-PointCloudComponent::PointCloudComponent() : GeometryComponent(GeometryComponentType::PointCloud)
-{
-}
-
-PointCloudComponent::~PointCloudComponent()
-{
-  this->clear();
-}
-
-GeometryComponent *PointCloudComponent::copy() const
-{
-  PointCloudComponent *new_component = new PointCloudComponent();
-  if (pointcloud_ != nullptr) {
-    new_component->pointcloud_ = BKE_pointcloud_copy_for_eval(pointcloud_, false);
-    new_component->ownership_ = GeometryOwnershipType::Owned;
-  }
-  return new_component;
-}
-
-void PointCloudComponent::clear()
-{
-  BLI_assert(this->is_mutable());
-  if (pointcloud_ != nullptr) {
-    if (ownership_ == GeometryOwnershipType::Owned) {
-      BKE_id_free(nullptr, pointcloud_);
-    }
-    pointcloud_ = nullptr;
-  }
-}
-
-bool PointCloudComponent::has_pointcloud() const
-{
-  return pointcloud_ != nullptr;
-}
-
-/* Clear the component and replace it with the new point cloud. */
-void PointCloudComponent::replace(PointCloud *pointcloud, GeometryOwnershipType ownership)
-{
-  BLI_assert(this->is_mutable());
-  this->clear();
-  pointcloud_ = pointcloud;
-  ownership_ = ownership;
-}
-
-/* Return the point cloud and clear the component. The caller takes over responsibility for freeing
- * the point cloud (if the component was responsible before). */
-PointCloud *PointCloudComponent::release()
-{
-  BLI_assert(this->is_mutable());
-  PointCloud *pointcloud = pointcloud_;
-  pointcloud_ = nullptr;
-  return pointcloud;
-}
-
-/* Get the point cloud from this component. This method can be used by multiple threads at the same
- * time. Therefore, the returned point cloud should not be modified. No ownership is transferred.
- */
-const PointCloud *PointCloudComponent::get_for_read() const
-{
-  return pointcloud_;
-}
-
-/* Get the point cloud from this component. This method can only be used when the component is
- * mutable, i.e. it is not shared. The returned point cloud can be modified. No ownership is
- * transferred. */
-PointCloud *PointCloudComponent::get_for_write()
-{
-  BLI_assert(this->is_mutable());
-  if (ownership_ == GeometryOwnershipType::ReadOnly) {
-    pointcloud_ = BKE_pointcloud_copy_for_eval(pointcloud_, false);
-    ownership_ = GeometryOwnershipType::Owned;
-  }
-  return pointcloud_;
-}
-
-bool PointCloudComponent::is_empty() const
-{
-  return pointcloud_ == nullptr;
-}
-
-/** \} */
-
-/* -------------------------------------------------------------------- */
-/** \name Instances Component
- * \{ */
-
-InstancesComponent::InstancesComponent() : GeometryComponent(GeometryComponentType::Instances)
-{
-}
-
-GeometryComponent *InstancesComponent::copy() const
-{
-  InstancesComponent *new_component = new InstancesComponent();
-  new_component->transforms_ = transforms_;
-  new_component->instanced_data_ = instanced_data_;
-  return new_component;
-}
-
-void InstancesComponent::clear()
-{
-  instanced_data_.clear();
-  transforms_.clear();
-}
-
-void InstancesComponent::add_instance(Object *object, float4x4 transform, const int id)
-{
-  InstancedData data;
-  data.type = INSTANCE_DATA_TYPE_OBJECT;
-  data.data.object = object;
-  this->add_instance(data, transform, id);
-}
-
-void InstancesComponent::add_instance(Collection *collection, float4x4 transform, const int id)
-{
-  InstancedData data;
-  data.type = INSTANCE_DATA_TYPE_COLLECTION;
-  data.data.collection = collection;
-  this->add_instance(data, transform, id);
-}
-
-void InstancesComponent::add_instance(InstancedData data, float4x4 transform, const int id)
-{
-  instanced_data_.append(data);
-  transforms_.append(transform);
-  ids_.append(id);
-}
-
-Span<InstancedData> InstancesComponent::instanced_data() const
-{
-  return instanced_data_;
-}
-
-Span<float4x4> InstancesComponent::transforms() const
-{
-  return transforms_;
-}
-
-Span<int> InstancesComponent::ids() const
-{
-  return ids_;
-}
-
-MutableSpan<float4x4> InstancesComponent::transforms()
-{
-  return transforms_;
-}
-
-int InstancesComponent::instances_amount() const
-{
-  const int size = instanced_data_.size();
-  BLI_assert(transforms_.size() == size);
-  return size;
-}
-
-bool InstancesComponent::is_empty() const
-{
-  return transforms_.size() == 0;
-}
-
-static blender::Array<int> generate_unique_instance_ids(Span<int> original_ids)
-{
-  using namespace blender;
-  Array<int> unique_ids(original_ids.size());
-
-  Set<int> used_unique_ids;
-  used_unique_ids.reserve(original_ids.size());
-  Vector<int> instances_with_id_collision;
-  for (const int instance_index : original_ids.index_range()) {
-    const int original_id = original_ids[instance_index];
-    if (used_unique_ids.add(original_id)) {
-      /* The original id has not been used by another instance yet. */
-      unique_ids[instance_index] = original_id;
-    }
-    else {
-      /* The original id of this instance collided with a previous instance, it needs to be looked
-       * at again in a second pass. Don't generate a new random id here, because this might collide
-       * with other existing ids. */
-      instances_with_id_collision.append(instance_index);
-    }
-  }
-
-  Map<int, RandomNumberGenerator> generator_by_original_id;
-  for (const int instance_index : instances_with_id_collision) {
-    const int original_id = original_ids[instance_index];
-    RandomNumberGenerator &rng = generator_by_original_id.lookup_or_add_cb(original_id, [&]() {
-      RandomNumberGenerator rng;
-      rng.seed_random(original_id);
-      return rng;
-    });
-
-    const int max_iteration = 100;
-    for (int iteration = 0;; iteration++) {
-      /* Try generating random numbers until an unused one has been found. */
-      const int random_id = rng.get_int32();
-      if (used_unique_ids.add(random_id)) {
-        /* This random id is not used by another instance. */
-        unique_ids[instance_index] = random_id;
-        break;
-      }
-      if (iteration == max_iteration) {
-        /* It seems to be very unlikely that we ever run into this case (assuming there are less
-         * than 2^30 instances). However, if that happens, it's better to use an id that is not
-         * unique than to be stuck in an infinite loop. */
-        unique_ids[instance_index] = original_id;
-        break;
-      }
-    }
-  }
-
-  return unique_ids;
-}
-
-blender::Span<int> InstancesComponent::almost_unique_ids() const
-{
-  std::lock_guard lock(almost_unique_ids_mutex_);
-  if (almost_unique_ids_.size() != ids_.size()) {
-    almost_unique_ids_ = generate_unique_instance_ids(ids_);
-  }
-  return almost_unique_ids_;
-}
-
-/** \} */
-
-/* -------------------------------------------------------------------- */
-/** \name Volume Component
- * \{ */
-
-VolumeComponent::VolumeComponent() : GeometryComponent(GeometryComponentType::Volume)
-{
-}
-
-VolumeComponent::~VolumeComponent()
-{
-  this->clear();
-}
-
-GeometryComponent *VolumeComponent::copy() const
-{
-  VolumeComponent *new_component = new VolumeComponent();
-  if (volume_ != nullptr) {
-    new_component->volume_ = BKE_volume_copy_for_eval(volume_, false);
-    new_component->ownership_ = GeometryOwnershipType::Owned;
-  }
-  return new_component;
-}
-
-void VolumeComponent::clear()
-{
-  BLI_assert(this->is_mutable());
-  if (volume_ != nullptr) {
-    if (ownership_ == GeometryOwnershipType::Owned) {
-      BKE_id_free(nullptr, volume_);
-    }
-    volume_ = nullptr;
-  }
-}
-
-bool VolumeComponent::has_volume() const
-{
-  return volume_ != nullptr;
-}
-
-/* Clear the component and replace it with the new volume. */
-void VolumeComponent::replace(Volume *volume, GeometryOwnershipType ownership)
-{
-  BLI_assert(this->is_mutable());
-  this->clear();
-  volume_ = volume;
-  ownership_ = ownership;
-}
-
-/* Return the volume and clear the component. The caller takes over responsibility for freeing the
- * volume (if the component was responsible before). */
-Volume *VolumeComponent::release()
-{
-  BLI_assert(this->is_mutable());
-  Volume *volume = volume_;
-  volume_ = nullptr;
-  return volume;
-}
-
-/* Get the volume from this component. This method can be used by multiple threads at the same
- * time. Therefore, the returned volume should not be modified. No ownership is transferred. */
-const Volume *VolumeComponent::get_for_read() const
-{
-  return volume_;
-}
-
-/* Get the volume from this component. This method can only be used when the component is mutable,
- * i.e. it is not shared. The returned volume can be modified. No ownership is transferred. */
-Volume *VolumeComponent::get_for_write()
-{
-  BLI_assert(this->is_mutable());
-  if (ownership_ == GeometryOwnershipType::ReadOnly) {
-    volume_ = BKE_volume_copy_for_eval(volume_, false);
-    ownership_ = GeometryOwnershipType::Owned;
-  }
-  return volume_;
-}
-
-/** \} */
-
-/* -------------------------------------------------------------------- */
 /** \name C API
  * \{ */