Cleanup: remove dead code
This commit is contained in:
parent
a9fc9ce0ae
commit
111a77e818
|
@ -54,7 +54,6 @@
|
|||
#include "BLI_map.hh"
|
||||
#include "BLT_translation.h"
|
||||
|
||||
#include "FN_attributes_ref.hh"
|
||||
#include "FN_multi_function_network_evaluation.hh"
|
||||
#include "FN_multi_function_network_optimization.hh"
|
||||
|
||||
|
|
|
@ -27,7 +27,6 @@ set(INC_SYS
|
|||
)
|
||||
|
||||
set(SRC
|
||||
intern/attributes_ref.cc
|
||||
intern/cpp_types.cc
|
||||
intern/multi_function.cc
|
||||
intern/multi_function_builder.cc
|
||||
|
@ -36,7 +35,6 @@ set(SRC
|
|||
intern/multi_function_network_optimization.cc
|
||||
|
||||
FN_array_spans.hh
|
||||
FN_attributes_ref.hh
|
||||
FN_cpp_type.hh
|
||||
FN_generic_pointer.hh
|
||||
FN_generic_value_map.hh
|
||||
|
@ -63,7 +61,6 @@ blender_add_lib(bf_functions "${SRC}" "${INC}" "${INC_SYS}" "${LIB}")
|
|||
if(WITH_GTESTS)
|
||||
set(TEST_SRC
|
||||
tests/FN_array_spans_test.cc
|
||||
tests/FN_attributes_ref_test.cc
|
||||
tests/FN_cpp_type_test.cc
|
||||
tests/FN_generic_vector_array_test.cc
|
||||
tests/FN_multi_function_network_test.cc
|
||||
|
|
|
@ -1,341 +0,0 @@
|
|||
/*
|
||||
* This program is free software; you can redistribute it and/or
|
||||
* modify it under the terms of the GNU General Public License
|
||||
* as published by the Free Software Foundation; either version 2
|
||||
* of the License, or (at your option) any later version.
|
||||
*
|
||||
* This program is distributed in the hope that it will be useful,
|
||||
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
* GNU General Public License for more details.
|
||||
*
|
||||
* You should have received a copy of the GNU General Public License
|
||||
* along with this program; if not, write to the Free Software Foundation,
|
||||
* Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
||||
*/
|
||||
|
||||
#pragma once
|
||||
|
||||
/** \file
|
||||
* \ingroup fn
|
||||
*
|
||||
* An AttributesRef references multiple arrays of equal length. Each array has a corresponding name
|
||||
* and index.
|
||||
*/
|
||||
|
||||
#include <optional>
|
||||
|
||||
#include "FN_spans.hh"
|
||||
|
||||
#include "BLI_linear_allocator.hh"
|
||||
#include "BLI_map.hh"
|
||||
#include "BLI_utility_mixins.hh"
|
||||
#include "BLI_vector_set.hh"
|
||||
|
||||
namespace blender::fn {
|
||||
|
||||
class AttributesInfo;
|
||||
|
||||
class AttributesInfoBuilder : NonCopyable, NonMovable {
|
||||
private:
|
||||
LinearAllocator<> allocator_;
|
||||
VectorSet<std::string> names_;
|
||||
Vector<const CPPType *> types_;
|
||||
Vector<void *> defaults_;
|
||||
|
||||
friend AttributesInfo;
|
||||
|
||||
public:
|
||||
AttributesInfoBuilder() = default;
|
||||
~AttributesInfoBuilder();
|
||||
|
||||
template<typename T> bool add(StringRef name, const T &default_value)
|
||||
{
|
||||
return this->add(name, CPPType::get<T>(), static_cast<const void *>(&default_value));
|
||||
}
|
||||
|
||||
bool add(StringRef name, const CPPType &type, const void *default_value = nullptr);
|
||||
};
|
||||
|
||||
/**
|
||||
* Stores which attributes are in an AttributesRef. Every attribute has a unique index, a unique
|
||||
* name, a type and a default value.
|
||||
*/
|
||||
class AttributesInfo : NonCopyable, NonMovable {
|
||||
private:
|
||||
LinearAllocator<> allocator_;
|
||||
Map<StringRefNull, int> index_by_name_;
|
||||
Vector<StringRefNull> name_by_index_;
|
||||
Vector<const CPPType *> type_by_index_;
|
||||
Vector<void *> defaults_;
|
||||
|
||||
public:
|
||||
AttributesInfo() = default;
|
||||
AttributesInfo(const AttributesInfoBuilder &builder);
|
||||
~AttributesInfo();
|
||||
|
||||
int size() const
|
||||
{
|
||||
return name_by_index_.size();
|
||||
}
|
||||
|
||||
IndexRange index_range() const
|
||||
{
|
||||
return name_by_index_.index_range();
|
||||
}
|
||||
|
||||
StringRefNull name_of(int index) const
|
||||
{
|
||||
return name_by_index_[index];
|
||||
}
|
||||
|
||||
int index_of(StringRef name) const
|
||||
{
|
||||
return index_by_name_.lookup_as(name);
|
||||
}
|
||||
|
||||
const void *default_of(int index) const
|
||||
{
|
||||
return defaults_[index];
|
||||
}
|
||||
|
||||
const void *default_of(StringRef name) const
|
||||
{
|
||||
return this->default_of(this->index_of(name));
|
||||
}
|
||||
|
||||
template<typename T> const T &default_of(int index) const
|
||||
{
|
||||
BLI_assert(type_by_index_[index]->is<T>());
|
||||
return *static_cast<T *>(defaults_[index]);
|
||||
}
|
||||
|
||||
template<typename T> const T &default_of(StringRef name) const
|
||||
{
|
||||
return this->default_of<T>(this->index_of(name));
|
||||
}
|
||||
|
||||
const CPPType &type_of(int index) const
|
||||
{
|
||||
return *type_by_index_[index];
|
||||
}
|
||||
|
||||
const CPPType &type_of(StringRef name) const
|
||||
{
|
||||
return this->type_of(this->index_of(name));
|
||||
}
|
||||
|
||||
bool has_attribute(StringRef name, const CPPType &type) const
|
||||
{
|
||||
return this->try_index_of(name, type) >= 0;
|
||||
}
|
||||
|
||||
int try_index_of(StringRef name) const
|
||||
{
|
||||
return index_by_name_.lookup_default_as(name, -1);
|
||||
}
|
||||
|
||||
int try_index_of(StringRef name, const CPPType &type) const
|
||||
{
|
||||
int index = this->try_index_of(name);
|
||||
if (index == -1) {
|
||||
return -1;
|
||||
}
|
||||
else if (this->type_of(index) == type) {
|
||||
return index;
|
||||
}
|
||||
else {
|
||||
return -1;
|
||||
}
|
||||
}
|
||||
};
|
||||
|
||||
/**
|
||||
* References multiple arrays that match the description of an AttributesInfo instance. This class
|
||||
* is supposed to be relatively cheap to copy. It does not own any of the arrays itself.
|
||||
*/
|
||||
class MutableAttributesRef {
|
||||
private:
|
||||
const AttributesInfo *info_;
|
||||
Span<void *> buffers_;
|
||||
IndexRange range_;
|
||||
|
||||
friend class AttributesRef;
|
||||
|
||||
public:
|
||||
MutableAttributesRef(const AttributesInfo &info, Span<void *> buffers, int64_t size)
|
||||
: MutableAttributesRef(info, buffers, IndexRange(size))
|
||||
{
|
||||
}
|
||||
|
||||
MutableAttributesRef(const AttributesInfo &info, Span<void *> buffers, IndexRange range)
|
||||
: info_(&info), buffers_(buffers), range_(range)
|
||||
{
|
||||
}
|
||||
|
||||
int64_t size() const
|
||||
{
|
||||
return range_.size();
|
||||
}
|
||||
|
||||
IndexRange index_range() const
|
||||
{
|
||||
return IndexRange(this->size());
|
||||
}
|
||||
|
||||
const AttributesInfo &info() const
|
||||
{
|
||||
return *info_;
|
||||
}
|
||||
|
||||
GMutableSpan get(int index) const
|
||||
{
|
||||
const CPPType &type = info_->type_of(index);
|
||||
void *ptr = POINTER_OFFSET(buffers_[index], type.size() * range_.start());
|
||||
return GMutableSpan(type, ptr, range_.size());
|
||||
}
|
||||
|
||||
GMutableSpan get(StringRef name) const
|
||||
{
|
||||
return this->get(info_->index_of(name));
|
||||
}
|
||||
|
||||
template<typename T> MutableSpan<T> get(int index) const
|
||||
{
|
||||
BLI_assert(info_->type_of(index).is<T>());
|
||||
return MutableSpan<T>(static_cast<T *>(buffers_[index]) + range_.start(), range_.size());
|
||||
}
|
||||
|
||||
template<typename T> MutableSpan<T> get(StringRef name) const
|
||||
{
|
||||
return this->get<T>(info_->index_of(name));
|
||||
}
|
||||
|
||||
std::optional<GMutableSpan> try_get(StringRef name, const CPPType &type) const
|
||||
{
|
||||
int index = info_->try_index_of(name, type);
|
||||
if (index == -1) {
|
||||
return {};
|
||||
}
|
||||
else {
|
||||
return this->get(index);
|
||||
}
|
||||
}
|
||||
|
||||
template<typename T> std::optional<MutableSpan<T>> try_get(StringRef name) const
|
||||
{
|
||||
int index = info_->try_index_of(name);
|
||||
if (index == -1) {
|
||||
return {};
|
||||
}
|
||||
else if (info_->type_of(index).is<T>()) {
|
||||
return this->get<T>(index);
|
||||
}
|
||||
else {
|
||||
return {};
|
||||
}
|
||||
}
|
||||
|
||||
MutableAttributesRef slice(IndexRange range) const
|
||||
{
|
||||
return this->slice(range.start(), range.size());
|
||||
}
|
||||
|
||||
MutableAttributesRef slice(int64_t start, int64_t size) const
|
||||
{
|
||||
return MutableAttributesRef(*info_, buffers_, range_.slice(start, size));
|
||||
}
|
||||
};
|
||||
|
||||
class AttributesRef {
|
||||
private:
|
||||
const AttributesInfo *info_;
|
||||
Span<const void *> buffers_;
|
||||
IndexRange range_;
|
||||
|
||||
public:
|
||||
AttributesRef(const AttributesInfo &info, Span<const void *> buffers, int64_t size)
|
||||
: AttributesRef(info, buffers, IndexRange(size))
|
||||
{
|
||||
}
|
||||
|
||||
AttributesRef(const AttributesInfo &info, Span<const void *> buffers, IndexRange range)
|
||||
: info_(&info), buffers_(buffers), range_(range)
|
||||
{
|
||||
}
|
||||
|
||||
AttributesRef(MutableAttributesRef attributes)
|
||||
: info_(attributes.info_), buffers_(attributes.buffers_), range_(attributes.range_)
|
||||
{
|
||||
}
|
||||
|
||||
int64_t size() const
|
||||
{
|
||||
return range_.size();
|
||||
}
|
||||
|
||||
const AttributesInfo &info() const
|
||||
{
|
||||
return *info_;
|
||||
}
|
||||
|
||||
GSpan get(int index) const
|
||||
{
|
||||
const CPPType &type = info_->type_of(index);
|
||||
const void *ptr = POINTER_OFFSET(buffers_[index], type.size() * range_.start());
|
||||
return GSpan(type, ptr, range_.size());
|
||||
}
|
||||
|
||||
GSpan get(StringRef name) const
|
||||
{
|
||||
return this->get(info_->index_of(name));
|
||||
}
|
||||
|
||||
template<typename T> Span<T> get(int index) const
|
||||
{
|
||||
BLI_assert(info_->type_of(index).is<T>());
|
||||
return Span<T>(static_cast<T *>(buffers_[index]) + range_.start(), range_.size());
|
||||
}
|
||||
|
||||
template<typename T> Span<T> get(StringRef name) const
|
||||
{
|
||||
return this->get<T>(info_->index_of(name));
|
||||
}
|
||||
|
||||
std::optional<GSpan> try_get(StringRef name, const CPPType &type) const
|
||||
{
|
||||
int64_t index = info_->try_index_of(name, type);
|
||||
if (index == -1) {
|
||||
return {};
|
||||
}
|
||||
else {
|
||||
return this->get(index);
|
||||
}
|
||||
}
|
||||
|
||||
template<typename T> std::optional<Span<T>> try_get(StringRef name) const
|
||||
{
|
||||
int index = info_->try_index_of(name);
|
||||
if (index == -1) {
|
||||
return {};
|
||||
}
|
||||
else if (info_->type_of(index).is<T>()) {
|
||||
return this->get<T>(index);
|
||||
}
|
||||
else {
|
||||
return {};
|
||||
}
|
||||
}
|
||||
|
||||
AttributesRef slice(IndexRange range) const
|
||||
{
|
||||
return this->slice(range.start(), range.size());
|
||||
}
|
||||
|
||||
AttributesRef slice(int64_t start, int64_t size) const
|
||||
{
|
||||
return AttributesRef(*info_, buffers_, range_.slice(start, size));
|
||||
}
|
||||
};
|
||||
|
||||
} // namespace blender::fn
|
|
@ -1,78 +0,0 @@
|
|||
/*
|
||||
* This program is free software; you can redistribute it and/or
|
||||
* modify it under the terms of the GNU General Public License
|
||||
* as published by the Free Software Foundation; either version 2
|
||||
* of the License, or (at your option) any later version.
|
||||
*
|
||||
* This program is distributed in the hope that it will be useful,
|
||||
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
* GNU General Public License for more details.
|
||||
*
|
||||
* You should have received a copy of the GNU General Public License
|
||||
* along with this program; if not, write to the Free Software Foundation,
|
||||
* Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
||||
*/
|
||||
|
||||
#include "FN_attributes_ref.hh"
|
||||
|
||||
namespace blender::fn {
|
||||
|
||||
AttributesInfoBuilder::~AttributesInfoBuilder()
|
||||
{
|
||||
for (int i : defaults_.index_range()) {
|
||||
types_[i]->destruct(defaults_[i]);
|
||||
}
|
||||
}
|
||||
|
||||
bool AttributesInfoBuilder::add(StringRef name, const CPPType &type, const void *default_value)
|
||||
{
|
||||
if (name.size() == 0) {
|
||||
std::cout << "Warning: Tried to add an attribute with empty name.\n";
|
||||
return false;
|
||||
}
|
||||
if (names_.add_as(name)) {
|
||||
types_.append(&type);
|
||||
|
||||
if (default_value == nullptr) {
|
||||
default_value = type.default_value();
|
||||
}
|
||||
void *dst = allocator_.allocate(type.size(), type.alignment());
|
||||
type.copy_to_uninitialized(default_value, dst);
|
||||
defaults_.append(dst);
|
||||
return true;
|
||||
}
|
||||
|
||||
const CPPType &stored_type = *types_[names_.index_of_as(name)];
|
||||
if (stored_type != type) {
|
||||
std::cout << "Warning: Tried to add an attribute twice with different types (" << name << ": "
|
||||
<< stored_type.name() << ", " << type.name() << ").\n";
|
||||
}
|
||||
return false;
|
||||
}
|
||||
|
||||
AttributesInfo::AttributesInfo(const AttributesInfoBuilder &builder)
|
||||
{
|
||||
for (int i : builder.types_.index_range()) {
|
||||
StringRefNull name = allocator_.copy_string(builder.names_[i]);
|
||||
const CPPType &type = *builder.types_[i];
|
||||
const void *default_value = builder.defaults_[i];
|
||||
|
||||
index_by_name_.add_new(name, i);
|
||||
name_by_index_.append(name);
|
||||
type_by_index_.append(&type);
|
||||
|
||||
void *dst = allocator_.allocate(type.size(), type.alignment());
|
||||
type.copy_to_uninitialized(default_value, dst);
|
||||
defaults_.append(dst);
|
||||
}
|
||||
}
|
||||
|
||||
AttributesInfo::~AttributesInfo()
|
||||
{
|
||||
for (int i : defaults_.index_range()) {
|
||||
type_by_index_[i]->destruct(defaults_[i]);
|
||||
}
|
||||
}
|
||||
|
||||
} // namespace blender::fn
|
|
@ -1,97 +0,0 @@
|
|||
/* Apache License, Version 2.0 */
|
||||
|
||||
#include "BLI_float3.hh"
|
||||
#include "FN_attributes_ref.hh"
|
||||
|
||||
#include "testing/testing.h"
|
||||
|
||||
namespace blender::fn::tests {
|
||||
|
||||
TEST(attributes_info, BuildEmpty)
|
||||
{
|
||||
AttributesInfoBuilder info_builder;
|
||||
AttributesInfo info{info_builder};
|
||||
|
||||
EXPECT_EQ(info.size(), 0);
|
||||
}
|
||||
|
||||
TEST(attributes_info, AddSameNameTwice)
|
||||
{
|
||||
AttributesInfoBuilder info_builder;
|
||||
info_builder.add<int>("A", 4);
|
||||
info_builder.add<int>("A", 5);
|
||||
AttributesInfo info{info_builder};
|
||||
EXPECT_EQ(info.size(), 1);
|
||||
EXPECT_TRUE(info.has_attribute("A", CPPType::get<int>()));
|
||||
EXPECT_FALSE(info.has_attribute("B", CPPType::get<int>()));
|
||||
EXPECT_FALSE(info.has_attribute("A", CPPType::get<float>()));
|
||||
EXPECT_EQ(info.default_of<int>("A"), 4);
|
||||
EXPECT_EQ(info.name_of(0), "A");
|
||||
EXPECT_EQ(info.index_range().start(), 0);
|
||||
EXPECT_EQ(info.index_range().one_after_last(), 1);
|
||||
}
|
||||
|
||||
TEST(attributes_info, BuildWithDefaultString)
|
||||
{
|
||||
AttributesInfoBuilder info_builder;
|
||||
info_builder.add("A", CPPType::get<std::string>());
|
||||
AttributesInfo info{info_builder};
|
||||
EXPECT_EQ(info.default_of<std::string>("A"), "");
|
||||
}
|
||||
|
||||
TEST(attributes_info, BuildWithGivenDefault)
|
||||
{
|
||||
AttributesInfoBuilder info_builder;
|
||||
info_builder.add<std::string>("A", "hello world");
|
||||
AttributesInfo info{info_builder};
|
||||
const void *default_value = info.default_of("A");
|
||||
EXPECT_EQ(*(const std::string *)default_value, "hello world");
|
||||
EXPECT_EQ(info.type_of("A"), CPPType::get<std::string>());
|
||||
}
|
||||
|
||||
TEST(mutable_attributes_ref, ComplexTest)
|
||||
{
|
||||
AttributesInfoBuilder info_builder;
|
||||
info_builder.add<float3>("Position", {0, 0, 10});
|
||||
info_builder.add<uint>("ID", 0);
|
||||
info_builder.add<float>("Size", 0.5f);
|
||||
info_builder.add<std::string>("Name", "<no name>");
|
||||
AttributesInfo info{info_builder};
|
||||
|
||||
int amount = 5;
|
||||
Array<float3> positions(amount);
|
||||
Array<uint> ids(amount, 0);
|
||||
Array<float> sizes(amount);
|
||||
Array<std::string> names(amount);
|
||||
|
||||
Array<void *> buffers = {positions.data(), ids.data(), sizes.data(), names.data()};
|
||||
MutableAttributesRef attributes{info, buffers, IndexRange(1, 3)};
|
||||
EXPECT_EQ(attributes.size(), 3);
|
||||
EXPECT_EQ(attributes.info().size(), 4);
|
||||
EXPECT_EQ(attributes.get("Position").data(), positions.data() + 1);
|
||||
EXPECT_EQ(attributes.get("ID").data(), ids.data() + 1);
|
||||
EXPECT_EQ(attributes.get("Size").data(), sizes.data() + 1);
|
||||
EXPECT_EQ(attributes.get("Name").data(), names.data() + 1);
|
||||
|
||||
EXPECT_EQ(attributes.get("ID").size(), 3);
|
||||
EXPECT_EQ(attributes.get<uint>("ID").size(), 3);
|
||||
|
||||
EXPECT_EQ(ids[2], 0);
|
||||
MutableSpan<uint> ids_span = attributes.get<uint>("ID");
|
||||
ids_span[1] = 42;
|
||||
EXPECT_EQ(ids[2], 42);
|
||||
|
||||
EXPECT_FALSE(attributes.try_get<int>("not existant").has_value());
|
||||
EXPECT_FALSE(attributes.try_get<int>("Position").has_value());
|
||||
EXPECT_TRUE(attributes.try_get<float3>("Position").has_value());
|
||||
EXPECT_FALSE(attributes.try_get("not existant", CPPType::get<int>()).has_value());
|
||||
EXPECT_FALSE(attributes.try_get("Position", CPPType::get<int>()).has_value());
|
||||
EXPECT_TRUE(attributes.try_get("Position", CPPType::get<float3>()).has_value());
|
||||
|
||||
MutableAttributesRef sliced = attributes.slice(IndexRange(1, 2));
|
||||
EXPECT_EQ(sliced.size(), 2);
|
||||
sliced.get<uint>("ID")[0] = 100;
|
||||
EXPECT_EQ(ids[2], 100);
|
||||
}
|
||||
|
||||
} // namespace blender::fn::tests
|
Loading…
Reference in New Issue