Repository navigation
Replies: 3 comments
|
As suggested by @Nordegraf, we should try to reduce dependency on boost. Below is a list of files with corresponding references. We should review and remove dependencies, where appropriate.
boost:
UG4 only: ./ugbase/common/util/factory.h:#include "../boost_serialization.h" ./ugbase/bridge/util_algebra_dependent.h:#include "common/util/end_boost_list.h" ./ugbase/lib_algebra/ordering_strategies/algorithms/ordering_algorithms.cpp:#include "boost_cuthill_mckee_ordering.cpp" ./ugbase/lib_algebra/ordering_strategies/algorithms/topological_ordering.cpp:#include "lib_algebra/graph_interface/sparsematrix_boost.h" not used: |
|
Many items have been addressed in 65ddbe4. The follwoing items remain: <boost/optional.h>: // std::optional is Cxx17 and not yet supported by all compilers, so we use boost::optional for now ;-( <boost/lexical_cast.hpp>: Could be replaced, e.g., by std::stoi, std::stod, etc.. However, extra care is required for not breaking any conversions. <boost/algorithm/string.hpp>: Occurs only once in ugcore/ugbase/lib_algebra/common/matrixio/matrix_io_mtx.h, where it is used for a string split (w/ blank delimiters). Replacement should be rather easy, but must be done with extra care. After this, only the following larger modules remain: <boost/archive/> |
|
Here are all dependencies from grep "<boost/" -r ugcore/ugbase/: ugcore/ugbase/registry/class.h:#include <boost/optional.hpp> // for boost::optional ugcore/ugbase/lib_algebra/common/matrixio/matrix_io_mtx.h:#include <boost/lexical_cast.hpp> ** ugcore/ugbase/lib_algebra/common/matrixio/matrix_io_mtx.h:#include <boost/algorithm/string.hpp> ** ugcore/ugbase/bridge/util_domain_dependent.h:#include <boost/mpl/if.hpp> |
Uh oh!
There was an error while loading. Please reload this page.
👋 Welcome!
We’re using Discussions as a place to connect with other members of our community. We hope that you:
build together 💪.
To get started, comment below with an introduction of yourself and tell us about what you do with this community.
All reactions