Implement bounding boxes with a separate class

[libs/math.git] / source / geometry / union.h

diff --git a/source/geometry/union.h b/source/geometry/union.h
index fadeff2ac9270dd71718aade2e733504e6a44e5b..8461f8484ee5d085560b6fb70f522447d9a95c6e 100644 (file)
--- a/source/geometry/union.h
+++ b/source/geometry/union.h
@@ -12,7 +12,7 @@ Joins component shapes together into one.
 template<typename T, unsigned D>
 struct UnionOps
 {
-       static HyperBox<T, D> combine_aabb(const HyperBox<T, D> &, const HyperBox<T, D> &);
+       static BoundingBox<T, D> combine_aabb(const BoundingBox<T, D> &a, const BoundingBox<T, D> &b) { return a|b; }
        static bool init_inside() { return false; }
        static bool combine_inside(bool a, bool b) { return a || b; }
        static bool is_inside_decided(bool a) { return a; }
@@ -23,9 +23,13 @@ struct UnionOps
 template<typename T, unsigned D>
 class Union: public CompositeShape<T, D, UnionOps<T, D> >
 {
+private:
+       Union() { }
 public:
        Union(const Shape<T, D> &, const Shape<T, D> &);
-       Union(const std::vector<Shape<T, D> *> &);
+
+       template<typename Iter>
+       static Union from_iterator_range(const Iter &, const Iter &);
 
        virtual Union *clone() const;
 };
@@ -36,24 +40,18 @@ inline Union<T, D>::Union(const Shape<T, D> &s1, const Shape<T, D> &s2):
 { }
 
 template<typename T, unsigned D>
-inline Union<T, D>::Union(const std::vector<Shape<T, D> *> &s):
-       CompositeShape<T, D, UnionOps<T, D> >(s)
-{ }
-
-template<typename T, unsigned D>
-inline Union<T, D> *Union<T, D>::clone() const
+template<typename Iter>
+inline Union<T, D> Union<T, D>::from_iterator_range(const Iter &begin, const Iter &end)
 {
-       return new Union<T, D>(this->shapes);
+       Union<T, D> shape;
+       shape.init_from_iter_range(begin, end);
+       return shape;
 }
 
-
 template<typename T, unsigned D>
-inline HyperBox<T, D> UnionOps<T, D>::combine_aabb(const HyperBox<T, D> &box1, const HyperBox<T, D> &box2)
+inline Union<T, D> *Union<T, D>::clone() const
 {
-       LinAl::Vector<T, D> dimensions;
-       for(unsigned i=0; i<D; ++i)
-               dimensions[i] = std::max(box1.get_dimension(i), box2.get_dimension(i));
-       return HyperBox<T, D>(dimensions);
+       return new Union<T, D>(*this);
 }
 
 } // namespace Geometry