Olena-patches September 2008

olena-patches@lrde.epita.fr

12 participants
359 discussions

2429: Help the compiler not to mix up classes and methods.

by Roland Levillain

* mln/topo/face.hh (mln::topo::face<D>::face_data): Rename method as... (mln::topo::face<D>::data): ...this. * mln/topo/n_face.hh (mln::topo::n_face<N, D>::face_data): Rename method as... (mln::topo::n_face<N, D>::data): ...this. * mln/topo/complex.hh (mln::topo::complex<D>::connect_): Adjust. * tests/topo/complex.cc: Adjust. --- milena/ChangeLog | 13 +++++++++++++ milena/mln/topo/complex.hh | 4 ++-- milena/mln/topo/face.hh | 4 ++-- milena/mln/topo/n_face.hh | 4 ++-- milena/tests/topo/complex.cc | 4 ++-- 5 files changed, 21 insertions(+), 8 deletions(-) diff --git a/milena/ChangeLog b/milena/ChangeLog index 19eda01..8f404c2 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,18 @@ 2008-09-29 Roland Levillain <roland(a)lrde.epita.fr> + Help the compiler not to mix up classes and methods. + + * mln/topo/face.hh (mln::topo::face<D>::face_data): + Rename method as... + (mln::topo::face<D>::data): ...this. + * mln/topo/n_face.hh (mln::topo::n_face<N, D>::face_data): + Rename method as... + (mln::topo::n_face<N, D>::data): ...this. + * mln/topo/complex.hh (mln::topo::complex<D>::connect_): Adjust. + * tests/topo/complex.cc: Adjust. + +2008-09-29 Roland Levillain <roland(a)lrde.epita.fr> + Help the compiler not to mix up typedefs and methods. * mln/topo/face_iter.hh, diff --git a/milena/mln/topo/complex.hh b/milena/mln/topo/complex.hh index 13e21d0..c16ec9b 100644 --- a/milena/mln/topo/complex.hh +++ b/milena/mln/topo/complex.hh @@ -572,8 +572,8 @@ namespace mln // Ensure N is compatible with D. metal::bool_< N <= D >::check(); - f1.face_data().connect_higher_dim_face(f2); - f2.face_data().connect_lower_dim_face(f1); + f1.data().connect_higher_dim_face(f2); + f2.data().connect_lower_dim_face(f1); } diff --git a/milena/mln/topo/face.hh b/milena/mln/topo/face.hh index b35f6c7..b2d82b1 100644 --- a/milena/mln/topo/face.hh +++ b/milena/mln/topo/face.hh @@ -95,7 +95,7 @@ namespace mln /// Return the mln::topo::face_data pointed by this handle. template <unsigned N> - face_data<N, D>& face_data() const; + face_data<N, D>& data() const; /// \} private: @@ -237,7 +237,7 @@ namespace mln template <unsigned N> inline face_data<N, D>& - face<D>::face_data() const + face<D>::data() const { mln_precondition(n_ == N); mln_precondition(is_valid()); diff --git a/milena/mln/topo/n_face.hh b/milena/mln/topo/n_face.hh index e243cc6..8d9664b 100644 --- a/milena/mln/topo/n_face.hh +++ b/milena/mln/topo/n_face.hh @@ -88,7 +88,7 @@ namespace mln void set_face_id(unsigned face_id); /// Return the mln::topo::face_data pointed by this handle. - face_data<N, D>& face_data() const; + face_data<N, D>& data() const; /// \} private: @@ -211,7 +211,7 @@ namespace mln template <unsigned N, unsigned D> inline face_data<N, D>& - n_face<N, D>::face_data() const + n_face<N, D>::data() const { mln_precondition(is_valid()); return cplx_->template face_data_<N>(face_id_); diff --git a/milena/tests/topo/complex.cc b/milena/tests/topo/complex.cc index 59d6047..782bf97 100644 --- a/milena/tests/topo/complex.cc +++ b/milena/tests/topo/complex.cc @@ -112,12 +112,12 @@ int main() `-------------------*/ // Get the face data from (``static'') face handle E0. - const topo::face_data<1, D>& face1 = e0.face_data(); + const topo::face_data<1, D>& face1 = e0.data(); // Any-face handle. topo::face<D> af(e0); // Get the face data from (``dynamic'') face handle AF. - const topo::face_data<1, D>& face2 = af.face_data<1>(); + const topo::face_data<1, D>& face2 = af.data<1>(); mln_assertion(&face1 == &face2); -- 1.5.6.5

16 years, 2 months

2428: Help the compiler not to mix up typedefs and methods.

by Roland Levillain

* mln/topo/face_iter.hh, * mln/topo/n_face_iter.hh, * mln/topo/static_n_face_iter.hh (mln::topo::face_fwd_iter<D>::face) (mln::topo::face_bkd_iter<D>::face) (mln::topo::n_face_fwd_iter<D>::face) (mln::topo::n_face_bkd_iter<D>::face) (mln::topo::static_n_face_fwd_iter<D>::face) (mln::topo::static_n_face_bkd_iter<D>::face): Remove typedef. (mln::topo::face_fwd_iter<D>::super_) (mln::topo::face_bkd_iter<D>::super_) (mln::topo::n_face_fwd_iter<D>::super_) (mln::topo::n_face_bkd_iter<D>::super_) (mln::topo::static_n_face_fwd_iter<D>::super_) (mln::topo::static_n_face_bkd_iter<D>::super_): Adjust. --- milena/ChangeLog | 22 ++++++++++++++++++++++ milena/mln/topo/face_iter.hh | 12 ++---------- milena/mln/topo/n_face_iter.hh | 12 ++---------- milena/mln/topo/static_n_face_iter.hh | 12 ++---------- 4 files changed, 28 insertions(+), 30 deletions(-) diff --git a/milena/ChangeLog b/milena/ChangeLog index f00feeb..19eda01 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,27 @@ 2008-09-29 Roland Levillain <roland(a)lrde.epita.fr> + Help the compiler not to mix up typedefs and methods. + + * mln/topo/face_iter.hh, + * mln/topo/n_face_iter.hh, + * mln/topo/static_n_face_iter.hh + (mln::topo::face_fwd_iter<D>::face) + (mln::topo::face_bkd_iter<D>::face) + (mln::topo::n_face_fwd_iter<D>::face) + (mln::topo::n_face_bkd_iter<D>::face) + (mln::topo::static_n_face_fwd_iter<D>::face) + (mln::topo::static_n_face_bkd_iter<D>::face): + Remove typedef. + (mln::topo::face_fwd_iter<D>::super_) + (mln::topo::face_bkd_iter<D>::super_) + (mln::topo::n_face_fwd_iter<D>::super_) + (mln::topo::n_face_bkd_iter<D>::super_) + (mln::topo::static_n_face_fwd_iter<D>::super_) + (mln::topo::static_n_face_bkd_iter<D>::super_): + Adjust. + +2008-09-29 Roland Levillain <roland(a)lrde.epita.fr> + * mln/core/faces_psite.hh: Include csdtlib for abort(). 2008-09-29 Nicolas Ballas <ballas(a)lrde.epita.fr> diff --git a/milena/mln/topo/face_iter.hh b/milena/mln/topo/face_iter.hh index e366441..4de87cc 100644 --- a/milena/mln/topo/face_iter.hh +++ b/milena/mln/topo/face_iter.hh @@ -59,13 +59,9 @@ namespace mln class face_fwd_iter : public internal::complex_iter_base< face<D>, face_fwd_iter<D> > { - public: - /// Type of associated face. - typedef face<D> face; - private: typedef face_fwd_iter<D> self_; - typedef internal::complex_iter_base< face, self_ > super_; + typedef internal::complex_iter_base< face<D>, self_ > super_; public: using super_::is_valid; @@ -103,13 +99,9 @@ namespace mln class face_bkd_iter : public internal::complex_iter_base< face<D>, face_bkd_iter<D> > { - public: - /// Type of associated face. - typedef face<D> face; - private: typedef face_bkd_iter<D> self_; - typedef internal::complex_iter_base< face, self_ > super_; + typedef internal::complex_iter_base< face<D>, self_ > super_; public: using super_::is_valid; diff --git a/milena/mln/topo/n_face_iter.hh b/milena/mln/topo/n_face_iter.hh index 1156707..c4291e2 100644 --- a/milena/mln/topo/n_face_iter.hh +++ b/milena/mln/topo/n_face_iter.hh @@ -62,13 +62,9 @@ namespace mln class n_face_fwd_iter : public internal::complex_iter_base< face<D>, n_face_fwd_iter<D> > { - public: - /// Type of associated face. - typedef face<D> face; - private: typedef n_face_fwd_iter<D> self_; - typedef internal::complex_iter_base< face, self_ > super_; + typedef internal::complex_iter_base< face<D>, self_ > super_; public: using super_::is_valid; @@ -114,13 +110,9 @@ namespace mln class n_face_bkd_iter : public internal::complex_iter_base< face<D>, n_face_bkd_iter<D> > { - public: - /// Type of associated face. - typedef face<D> face; - private: typedef n_face_bkd_iter<D> self_; - typedef internal::complex_iter_base< face, self_ > super_; + typedef internal::complex_iter_base< face<D>, self_ > super_; public: using super_::is_valid; diff --git a/milena/mln/topo/static_n_face_iter.hh b/milena/mln/topo/static_n_face_iter.hh index 2146310..291b699 100644 --- a/milena/mln/topo/static_n_face_iter.hh +++ b/milena/mln/topo/static_n_face_iter.hh @@ -56,13 +56,9 @@ namespace mln : public internal::complex_iter_base< face<D>, static_n_face_fwd_iter<N, D> > { - public: - /// Type of associated face. - typedef face<D> face; - private: typedef static_n_face_fwd_iter<N, D> self_; - typedef internal::complex_iter_base< face, self_ > super_; + typedef internal::complex_iter_base< face<D>, self_ > super_; public: using super_::is_valid; @@ -103,13 +99,9 @@ namespace mln : public internal::complex_iter_base< face<D>, static_n_face_bkd_iter<N, D> > { - public: - /// Type of associated face. - typedef face<D> face; - private: typedef static_n_face_bkd_iter<N, D> self_; - typedef internal::complex_iter_base< face, self_ > super_; + typedef internal::complex_iter_base< face<D>, self_ > super_; public: using super_::is_valid; -- 1.5.6.5

16 years, 2 months

2427: * mln/core/faces_psite.hh: Include csdtlib for abort().

by Roland Levillain

--- milena/ChangeLog | 4 ++++ milena/mln/core/faces_psite.hh | 2 ++ 2 files changed, 6 insertions(+), 0 deletions(-) diff --git a/milena/ChangeLog b/milena/ChangeLog index b79e5d1..f00feeb 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,3 +1,7 @@ +2008-09-29 Roland Levillain <roland(a)lrde.epita.fr> + + * mln/core/faces_psite.hh: Include csdtlib for abort(). + 2008-09-29 Nicolas Ballas <ballas(a)lrde.epita.fr> Update documentation about the image value properties. diff --git a/milena/mln/core/faces_psite.hh b/milena/mln/core/faces_psite.hh index 0e4d635..d4c6913 100644 --- a/milena/mln/core/faces_psite.hh +++ b/milena/mln/core/faces_psite.hh @@ -32,6 +32,8 @@ /// \brief Definition of a point site based on the n-faces of a /// complex. +# include <cstdlib> + # include <mln/core/internal/pseudo_site_base.hh> # include <mln/topo/complex.hh> -- 1.5.6.5

16 years, 2 months

cleanup-2008 2425: Update documentation about the image value properties.

by Nicolas Ballas

https://svn.lrde.epita.fr/svn/oln/branches/cleanup-2008/milena Index: ChangeLog from Nicolas Ballas <ballas(a)lrde.epita.fr> Update documentation about the image value properties. * doc/technical/designs/properties/values.txt: Update. values.txt | 65 ++++++++++++++++++++++++++++++------------------------------- 1 file changed, 32 insertions(+), 33 deletions(-) Index: doc/technical/designs/properties/values.txt --- doc/technical/designs/properties/values.txt (revision 2425) +++ doc/technical/designs/properties/values.txt (working copy) @@ -3,12 +3,12 @@ Values related properties -* TODO - -> Are the properties related to the values sufficiant? - -> What are the interface related to each value property? +* What is this file about + -> Are the image properties about the values sufficient/correct? + -> What are the piece of interface related to each property? -> How these values properties affect the fill/paste/transform process? -* Values properties list: +* Image values properties: ** value_access: /any/ | @@ -33,12 +33,12 @@ value_storage == singleton => ima.value() -> return the image value - value_storage == one_block + value_storage == one_block (what about the fastest image?) => image.nelements() -> return the number of elements in memory => image.buffer() -> return the start adress of the ima buffer value_storage == piece_wise - //FIXME + // runs iterator? ** value_browsing:/any/ | @@ -55,7 +55,7 @@ => ima.values_taken() -> return the current values taken by ima => ima.change_value(oldVal, newVal) -> change the value oldVal to newVal - => ima.change_values(f) -> apply f (a function v2v) to all the + => ima.apply(f) -> apply f (a function v2v) to all the values => ima.sites(v) -> return the sites associated to the value v @@ -68,9 +68,11 @@ +* Impact on the fill and paste algorithms +** fill -* fill (depends on the value_storage property) +Fill an image ima with a value v. *** values_storage == oneblock // Replace by Memcopy? @@ -78,12 +80,10 @@ mln_value(I)* ptr = ima.buffer(); for (unsigned i = 0; i < n; ++i) *ptr++ = val; - **** Method related -> nelements -> buffer - *** value_storage == piece_wise // MemCopy on the runs? for (ungsigned i = 0; i < ima.nruns(); ++i) @@ -91,23 +91,37 @@ **** Method related: nruns ??? - -*** value_storage == any && value_browsing == value_wise - +*** value_browsing == value_wise // direct access to the value mln_viter(mln_taken_values(I)) viter(I.taken_values(); for_all(viter); ima.changes_values(viter, v); - **** Method related: -> iterator on value taken -> change_values -*** Generic case: all the other case +*** generic case mln_piter(I) p(ima.domain()); for_all(p) ima(p) = v; +** Paste + +Paste value data from an image source to destination. + +*** values_storage == oneblock (for source an destination) +// Replace by Memcopy? +const unsigned n = ima.nelements(); +mln_value(I)* ptr_src = source.buffer(); +mln_value(I)* ptr_dest = destination.buffer(); +for (unsigned i = 0; i < n; ++i) + *ptr_src++ = *ptr_dest++; + + +*** generic case +mln_piter(I) p(source.domain()); +for_all(p) + destination(p) = source(p); * To keep in Mind @@ -147,22 +161,7 @@ -differentiate the localization of an image and its storage method. -* Image related with 'special' values access - -** Run Images (sparce, value_encoded) - -=> fill: *cpt++ = v sur chaque run -| -+-> related to memory storage - -** Pkey Images - -=> change_value - - - -* Note +** Image related with 'special' values access -- Cell access and Value access wise property depends on the image - --> property related: - cell-wise and value wise +*** Run Images (sparse, value_encoded) +*** Pkey Images

16 years, 2 months

2425: * tests/topo/complex.cc (test_static_n_face_iter): Aesthetic changes.

by Roland Levillain

--- milena/ChangeLog | 4 ++++ milena/tests/topo/complex.cc | 8 ++++---- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/milena/ChangeLog b/milena/ChangeLog index d5309e4..44b8b7a 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,9 @@ 2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + * tests/topo/complex.cc (test_static_n_face_iter): Aesthetic changes. + +2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Add iterators on n-faces (with a dynamically known `n'). * mln/topo/n_face_iter.hh: New. diff --git a/milena/tests/topo/complex.cc b/milena/tests/topo/complex.cc index 16099f5..59d6047 100644 --- a/milena/tests/topo/complex.cc +++ b/milena/tests/topo/complex.cc @@ -321,10 +321,10 @@ test_static_n_face_iter(topo::complex<D>& c) /* FIXME: Provide sugar. Maybe redefined mln_fwd_fiter and mln_bkd_fiter so that they expand as complex_faces_iters (instead of faces_iters). */ - topo::static_n_face_fwd_iter<N, D> fwd_ncf(c); - topo::static_n_face_bkd_iter<N, D> bkd_ncf(c); - for_all_2(fwd_ncf, bkd_ncf) - std::cout << fwd_ncf << ' ' << bkd_ncf << std::endl; + topo::static_n_face_fwd_iter<N, D> fwd_nf(c); + topo::static_n_face_bkd_iter<N, D> bkd_nf(c); + for_all_2(fwd_nf, bkd_nf) + std::cout << fwd_nf << ' ' << bkd_nf << std::endl; std::cout << std::endl; } -- 1.6.0.1

16 years, 3 months

2424: Add iterators on n-faces (with a dynamically known `n').

by Roland Levillain

* mln/topo/n_face_iter.hh: New. * mln/topo/complex.hh: Include mln/topo/n_face_iter.hh. * tests/topo/complex.cc: Exercise n_face_fwd_iter and n_face_bkd_iter. --- milena/ChangeLog | 9 ++ milena/mln/topo/complex.hh | 1 + milena/mln/topo/n_face_iter.hh | 311 ++++++++++++++++++++++++++++++++++++++++ milena/tests/topo/complex.cc | 33 +++-- 4 files changed, 345 insertions(+), 9 deletions(-) create mode 100644 milena/mln/topo/n_face_iter.hh diff --git a/milena/ChangeLog b/milena/ChangeLog index 4d9412b..d5309e4 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,14 @@ 2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Add iterators on n-faces (with a dynamically known `n'). + + * mln/topo/n_face_iter.hh: New. + * mln/topo/complex.hh: Include mln/topo/n_face_iter.hh. + * tests/topo/complex.cc: Exercise n_face_fwd_iter and + n_face_bkd_iter. + +2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Revive iterators on n-faces (with a statically known `n'). * mln/topo/attic/complex_faces_iter.hh: Rename as... diff --git a/milena/mln/topo/complex.hh b/milena/mln/topo/complex.hh index 61ce84b..13e21d0 100644 --- a/milena/mln/topo/complex.hh +++ b/milena/mln/topo/complex.hh @@ -49,6 +49,7 @@ # include <mln/topo/n_faces_set.hh> # include <mln/topo/face_iter.hh> +# include <mln/topo/n_face_iter.hh> # include <mln/topo/static_n_face_iter.hh> // FIXME: Disabled (moved to the attic). # if 0 diff --git a/milena/mln/topo/n_face_iter.hh b/milena/mln/topo/n_face_iter.hh new file mode 100644 index 0000000..1156707 --- /dev/null +++ b/milena/mln/topo/n_face_iter.hh @@ -0,0 +1,311 @@ +// Copyright (C) 2008 EPITA Research and Development Laboratory (LRDE) +// +// This file is part of the Olena Library. This library is free +// software; you can redistribute it and/or modify it under the terms +// of the GNU General Public License version 2 as published by the +// Free Software Foundation. +// +// This library 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 library; see the file COPYING. If not, write to +// the Free Software Foundation, 51 Franklin Street, Fifth Floor, +// Boston, MA 02111-1307, USA. +// +// As a special exception, you may use this file as part of a free +// software library without restriction. Specifically, if other files +// instantiate templates or use macros or inline functions from this +// file, or you compile this file and link it with other files to +// produce an executable, this file does not by itself cause the +// resulting executable to be covered by the GNU General Public +// License. +// reasons why the executable file might be covered by the GNU General +// Public License. + +#ifndef MLN_TOPO_N_FACE_ITER_HH +# define MLN_TOPO_N_FACE_ITER_HH + +/// \file mln/topo/n_face_iter.hh +/// \brief Definition of forward and backward iterators on all the +/// \a n-faces of a complex, \a n being a dynamic value. + +# include <mln/topo/internal/complex_iter_base.hh> +# include <mln/topo/face.hh> + +// FIXME: Factor a bit more? (Using complex_iter_base.) + +// FIXME: Rename the old commplex_faces_{fwd,bkd}_iter as +// static_n_face_{fwd,bkd}_iter. + + +namespace mln +{ + + namespace topo + { + + // Forward declaration. + template <unsigned D> class complex; + + + /*---------------------------. + | topo::n_face_fwd_iter<D>. | + `---------------------------*/ + + /// \brief Forward iterator on all the faces of an mln::complex<D>. + /// + /// \arg \p D The dimension of the complex this iterator belongs to. + template <unsigned D> + class n_face_fwd_iter + : public internal::complex_iter_base< face<D>, n_face_fwd_iter<D> > + { + public: + /// Type of associated face. + typedef face<D> face; + + private: + typedef n_face_fwd_iter<D> self_; + typedef internal::complex_iter_base< face, self_ > super_; + + public: + using super_::is_valid; + using super_::invalidate; + + public: + /// Construction and assignment. + /// \{ + n_face_fwd_iter(); + // FIXME: See comment in internal::complex_iter_base's default ctor + n_face_fwd_iter(complex<D>& c, unsigned n); + /// \} + + /// Manipulation. + /// \{ + /// Test if the iterator is valid. + void start(); + /// Go to the next point. + void next_(); + /// \} + + /// \brief Accessors. + /// + /// Shortcuts to face_'s accessors. + /// \{ + unsigned n() const; + void set_n (unsigned n); + /// \} + + private: + using super_::face_; + }; + + + /*---------------------------. + | topo::n_face_bkd_iter<D>. | + `---------------------------*/ + + /// \brief Backward iterator on all the faces of an mln::complex<D>. + /// + /// \arg \p D The dimension of the complex this iterator belongs to. + template <unsigned D> + class n_face_bkd_iter + : public internal::complex_iter_base< face<D>, n_face_bkd_iter<D> > + { + public: + /// Type of associated face. + typedef face<D> face; + + private: + typedef n_face_bkd_iter<D> self_; + typedef internal::complex_iter_base< face, self_ > super_; + + public: + using super_::is_valid; + using super_::invalidate; + + public: + /// Construction and assignment. + /// \{ + n_face_bkd_iter(); + // FIXME: See comment in internal::complex_iter_base's default ctor + n_face_bkd_iter(complex<D>& c, unsigned n); + /// \} + + /// Manipulation. + /// \{ + /// Start an iteration. + void start(); + /// Go to the next point. + void next_(); + /// \} + + /// \brief Accessors. + /// + /// Shortcuts to face_'s accessors. + /// \{ + unsigned n() const; + void set_n (unsigned n); + /// \} + + private: + using super_::face_; + }; + + + +# ifndef MLN_INCLUDE_ONLY + + /*---------------------------. + | topo::n_face_fwd_iter<D>. | + `---------------------------*/ + + template <unsigned D> + inline + n_face_fwd_iter<D>::n_face_fwd_iter() + : super_() + { + } + + template <unsigned D> + inline + n_face_fwd_iter<D>::n_face_fwd_iter(complex<D>& c, unsigned n) + : super_(c) + { + mln_precondition(n <= D); + set_cplx(c); + set_n(n); + mln_postcondition(!is_valid()); + } + + template <unsigned D> + inline + void + n_face_fwd_iter<D>::start() + { + face_.set_face_id(0u); + } + + template <unsigned D> + inline + void + n_face_fwd_iter<D>::next_() + { + if (is_valid()) + { + unsigned face_id = face_.face_id(); + // The number of faces of dimension N in cplx_. + unsigned nn_faces = face_.cplx().nfaces(n()); + if (face_id + 1 < nn_faces) + /* FIXME: Provide accessor face::face_id() returning + a mutable reference? This way, we could just write + + ++face_.face_id(); + + instead of the following. + + Or add {inc,add}_face_id() services. */ + face_.set_face_id(face_id + 1); + else + invalidate(); + } + } + + template <unsigned D> + inline + unsigned + n_face_fwd_iter<D>::n() const + { + return face_.n(); + } + + template <unsigned D> + inline + void + n_face_fwd_iter<D>::set_n(unsigned n) + { + mln_precondition(n <= D); + face_.set_n(n); + } + + + /*---------------------------. + | topo::n_face_bkd_iter<D>. | + `---------------------------*/ + + template <unsigned D> + inline + n_face_bkd_iter<D>::n_face_bkd_iter() + : super_() + { + } + + template <unsigned D> + inline + n_face_bkd_iter<D>::n_face_bkd_iter(complex<D>& c, unsigned n) + : super_(c) + { + mln_precondition(n <= D); + set_cplx(c); + set_n(n); + mln_postcondition(!is_valid()); + } + + template <unsigned D> + inline + void + n_face_bkd_iter<D>::start() + { + face_.set_n(n()); + face_.set_face_id(face_.cplx().nfaces(n()) - 1); + } + + template <unsigned D> + inline + void + n_face_bkd_iter<D>::next_() + { + if (is_valid()) + { + unsigned face_id = face_.face_id(); + if (face_id > 0) + /* FIXME: Provide accessor face::face_id() returning + a mutable reference? This way, we could just write + + ++face_.face_id(); + + instead of the following. + + Or add {inc,add}_face_id() services. */ + face_.set_face_id(face_id - 1); + else + invalidate(); + } + } + + template <unsigned D> + inline + unsigned + n_face_bkd_iter<D>::n() const + { + return face_.n(); + } + + template <unsigned D> + inline + void + n_face_bkd_iter<D>::set_n(unsigned n) + { + mln_precondition(n <= D); + face_.set_n(n); + } + +# endif // ! MLN_INCLUDE_ONLY + + } // end of namespace mln::topo + +} // end of namespace mln + +#endif // ! MLN_TOPO_N_FACE_ITER_HH diff --git a/milena/tests/topo/complex.cc b/milena/tests/topo/complex.cc index 4d220e7..16099f5 100644 --- a/milena/tests/topo/complex.cc +++ b/milena/tests/topo/complex.cc @@ -148,6 +148,18 @@ int main() actual complex processing since they are not really flexible --- but I'm not sure. */ + // Dynamic version. + for (unsigned n = 0; n <= D; ++n) + { + topo::n_face_fwd_iter<D> fwd_nf(c, n); + topo::n_face_fwd_iter<D> bkd_nf(c, n); + std::cout << "test (dynamic) n_face_iters (n = " << n << "):" + << std::endl; + for_all_2(fwd_nf, bkd_nf) + std::cout << fwd_nf << ' ' << bkd_nf << std::endl; + std::cout << std::endl; + } + // Static version. test_static_n_face_iter<0>(c); test_static_n_face_iter<1>(c); @@ -200,15 +212,21 @@ int main() ----------------------------------------------------------------- Name Definition ----------------------------------------------------------------- - complex<D> General complex + complex<D> General complex. - face_data<N, D> Face data - n_face<N, D> (Static) n-face handle - n_faces_set<N, D> Set of face handles - face<D> Dynamic face handle + face_data<N, D> Face data. + n_face<N, D> (Static) n-face handle. + n_faces_set<N, D> Set of face handles. + face<D> Dynamic face handle. face_fwd_iter<D>(c) | Iterators on all the faces - face_bkd_iter<D>(c) | of c + face_bkd_iter<D>(c) | of c. + + n_face_fwd_iter<D>(c) | Iterators on n-faces, + n_face_bkd_iter<D>(c) | n being dynamic. + + static_n_face_fwd_iter<D>(c) | Iterators on n-faces, + static_n_face_bkd_iter<D>(c) | n being static. internal::complex_iter_base<F, E> Factoring base class. ----------------------------------------------------------------- @@ -224,9 +242,6 @@ int main() ----------------------------------------------------------------- Name Definition ----------------------------------------------------------------- - n_faces_fwd_iter<D>(c) | Iterators on n-faces, - n_faces_bkd_iter<D>(c) | n being dynamic - adj_lower_faces_fwd_iter<D>(c, f) | Iterators on the adjacent adj_lower_faces_bkd_iter<D>(c, f) | (lower) (n-1)-faces of the | n-face f of the complex c, -- 1.6.0.1

16 years, 3 months

2423: Revive iterators on n-faces (with a statically known `n').

by Roland Levillain

* mln/topo/attic/complex_faces_iter.hh: Rename as... * mln/topo/static_n_face_iter.hh: ...this. (mln::topo::complex_faces_fwd_iter_<N, D>) (mln::topo::complex_faces_bkd_iter_<N, D>): Rename as... (static_n_face_fwd_iter) (static_n_face_bkd_iter): ...these. Adjust file layout. * mln/topo/complex.hh: Include mln/topo/static_n_face_iter.hh. * tests/topo/complex.cc (test_complex_faces_iter): Rename as... (test_static_n_face_iter): ...this. Adjust to static_n_face_fwd_iter and static_n_face_bkd_iter. (main): Exercise static_n_face_fwd_iter and static_n_face_bkd_iter. (test_faces_iter): Really disable this funcion. --- milena/ChangeLog | 21 +++++ milena/mln/topo/complex.hh | 2 +- ...complex_faces_iter.hh => static_n_face_iter.hh} | 86 ++++++++++---------- milena/tests/topo/complex.cc | 54 +++++++------ 4 files changed, 94 insertions(+), 69 deletions(-) rename milena/mln/topo/{attic/complex_faces_iter.hh => static_n_face_iter.hh} (72%) diff --git a/milena/ChangeLog b/milena/ChangeLog index d469c21..4d9412b 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,26 @@ 2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Revive iterators on n-faces (with a statically known `n'). + + * mln/topo/attic/complex_faces_iter.hh: Rename as... + * mln/topo/static_n_face_iter.hh: ...this. + (mln::topo::complex_faces_fwd_iter_<N, D>) + (mln::topo::complex_faces_bkd_iter_<N, D>): + Rename as... + (static_n_face_fwd_iter) + (static_n_face_bkd_iter): + ...these. + Adjust file layout. + * mln/topo/complex.hh: Include mln/topo/static_n_face_iter.hh. + * tests/topo/complex.cc (test_complex_faces_iter): Rename as... + (test_static_n_face_iter): ...this. + Adjust to static_n_face_fwd_iter and static_n_face_bkd_iter. + (main): Exercise static_n_face_fwd_iter and + static_n_face_bkd_iter. + (test_faces_iter): Really disable this funcion. + +2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Adjust paths of included files in the attic. * mln/core/site_set/attic/p_complex_faces_piter.hh, diff --git a/milena/mln/topo/complex.hh b/milena/mln/topo/complex.hh index e49f513..61ce84b 100644 --- a/milena/mln/topo/complex.hh +++ b/milena/mln/topo/complex.hh @@ -49,10 +49,10 @@ # include <mln/topo/n_faces_set.hh> # include <mln/topo/face_iter.hh> +# include <mln/topo/static_n_face_iter.hh> // FIXME: Disabled (moved to the attic). # if 0 # include <mln/topo/faces_iter.hh> -# include <mln/topo/complex_faces_iter.hh> # endif namespace mln diff --git a/milena/mln/topo/attic/complex_faces_iter.hh b/milena/mln/topo/static_n_face_iter.hh similarity index 72% rename from milena/mln/topo/attic/complex_faces_iter.hh rename to milena/mln/topo/static_n_face_iter.hh index c0d278c..2146310 100644 --- a/milena/mln/topo/attic/complex_faces_iter.hh +++ b/milena/mln/topo/static_n_face_iter.hh @@ -25,20 +25,16 @@ // reasons why the executable file might be covered by the GNU General // Public License. -#ifndef MLN_TOPO_COMPLEX_FACES_ITER_HH -# define MLN_TOPO_COMPLEX_FACES_ITER_HH +#ifndef MLN_TOPO_STATIC_N_FACE_ITER_HH +# define MLN_TOPO_STATIC_N_FACE_ITER_HH -/// \file mln/topo/complex_faces_iter.hh -/// \brief Definition of forward and backward iterators on the n-faces -/// of a complex. +/// \file mln/topo/static_n_face_iter.hh +/// \brief Definition of forward and backward iterators on all the +/// \p N-faces of a complex, \p N being a static value. # include <mln/topo/internal/complex_iter_base.hh> # include <mln/topo/face.hh> -/* FIXME: Should we drop N from the signature of these iterators? - I.e., have N be dynamic? Or better: provide other iterators where - N is dynamic? */ - namespace mln { @@ -46,25 +42,26 @@ namespace mln namespace topo { - /*--------------------------------------. - | topo::complex_faces_fwd_iter_<N, D>. | - `--------------------------------------*/ + /*-------------------------------------. + | topo::static_n_face_fwd_iter<N, D>. | + `-------------------------------------*/ - /// \brief Forward iterator on all the faces of a mln::complex<D>. + /// \brief Forward iterator on all the \p N-faces of a + /// mln::complex<D>. /// /// \arg \p N The dimension of the face associated to this iterator. /// \arg \p D The dimension of the complex this iterator belongs to. template <unsigned N, unsigned D> - class complex_faces_fwd_iter_ + class static_n_face_fwd_iter : public internal::complex_iter_base< face<D>, - complex_faces_fwd_iter_<N, D> > + static_n_face_fwd_iter<N, D> > { public: /// Type of associated face. typedef face<D> face; private: - typedef complex_faces_fwd_iter_<N, D> self_; + typedef static_n_face_fwd_iter<N, D> self_; typedef internal::complex_iter_base< face, self_ > super_; public: @@ -74,9 +71,9 @@ namespace mln public: /// Construction and assignment. /// \{ - complex_faces_fwd_iter_(); + static_n_face_fwd_iter(); // FIXME: See comment in internal::complex_iter_base's default ctor - complex_faces_fwd_iter_(complex<D>& c); + static_n_face_fwd_iter(complex<D>& c); /// \} /// Manipulation. @@ -92,25 +89,26 @@ namespace mln }; - /*--------------------------------------. - | topo::complex_faces_bkd_iter_<N, D>. | - `--------------------------------------*/ + /*-------------------------------------. + | topo::static_n_face_bkd_iter<N, D>. | + `-------------------------------------*/ - /// \brief Backward iterator on all the faces of a mln::complex<D>. + /// \brief Backward iterator on all the \p N-faces of a + /// mln::complex<D>. /// /// \arg \p N The dimension of the face associated to this iterator. /// \arg \p D The dimension of the complex this iterator belongs to. template <unsigned N, unsigned D> - class complex_faces_bkd_iter_ + class static_n_face_bkd_iter : public internal::complex_iter_base< face<D>, - complex_faces_bkd_iter_<N, D> > + static_n_face_bkd_iter<N, D> > { public: /// Type of associated face. typedef face<D> face; private: - typedef complex_faces_bkd_iter_<N, D> self_; + typedef static_n_face_bkd_iter<N, D> self_; typedef internal::complex_iter_base< face, self_ > super_; public: @@ -120,9 +118,9 @@ namespace mln public: /// Construction and assignment. /// \{ - complex_faces_bkd_iter_(); + static_n_face_bkd_iter(); // FIXME: See comment in internal::complex_iter_base's default ctor - complex_faces_bkd_iter_(complex<D>& c); + static_n_face_bkd_iter(complex<D>& c); /// \} /// Manipulation. @@ -141,13 +139,13 @@ namespace mln # ifndef MLN_INCLUDE_ONLY - /*--------------------------------------. - | topo::complex_faces_fwd_iter_<N, D>. | - `--------------------------------------*/ + /*-------------------------------------. + | topo::static_n_face_fwd_iter<N, D>. | + `-------------------------------------*/ template <unsigned N, unsigned D> inline - complex_faces_fwd_iter_<N, D>::complex_faces_fwd_iter_() + static_n_face_fwd_iter<N, D>::static_n_face_fwd_iter() : super_() { // Ensure N is compatible with D. @@ -158,7 +156,7 @@ namespace mln template <unsigned N, unsigned D> inline - complex_faces_fwd_iter_<N, D>::complex_faces_fwd_iter_(complex<D>& c) + static_n_face_fwd_iter<N, D>::static_n_face_fwd_iter(complex<D>& c) : super_(c) { // Ensure N is compatible with D. @@ -170,7 +168,7 @@ namespace mln template <unsigned N, unsigned D> inline void - complex_faces_fwd_iter_<N, D>::start() + static_n_face_fwd_iter<N, D>::start() { face_.set_face_id(0u); } @@ -178,12 +176,14 @@ namespace mln template <unsigned N, unsigned D> inline void - complex_faces_fwd_iter_<N, D>::next_() + static_n_face_fwd_iter<N, D>::next_() { if (is_valid()) { unsigned face_id = face_.face_id(); - if (face_id + 1 < face_.cplx().template nfaces<N>()) + // The number of faces of dimension N in cplx_. + unsigned nn_faces = face_.cplx().template nfaces<N>(); + if (face_id + 1 < nn_faces) /* FIXME: Provide accessor face::face_id() returning a mutable reference? This way, we could just write @@ -200,13 +200,13 @@ namespace mln } - /*--------------------------------------. - | topo::complex_faces_bkd_iter_<N, D>. | - `--------------------------------------*/ + /*-------------------------------------. + | topo::static_n_face_bkd_iter<N, D>. | + `-------------------------------------*/ template <unsigned N, unsigned D> inline - complex_faces_bkd_iter_<N, D>::complex_faces_bkd_iter_() + static_n_face_bkd_iter<N, D>::static_n_face_bkd_iter() : super_() { // Ensure N is compatible with D. @@ -217,7 +217,7 @@ namespace mln template <unsigned N, unsigned D> inline - complex_faces_bkd_iter_<N, D>::complex_faces_bkd_iter_(complex<D>& c) + static_n_face_bkd_iter<N, D>::static_n_face_bkd_iter(complex<D>& c) : super_(c) { // Ensure N is compatible with D. @@ -229,7 +229,7 @@ namespace mln template <unsigned N, unsigned D> inline void - complex_faces_bkd_iter_<N, D>::start() + static_n_face_bkd_iter<N, D>::start() { face_.set_face_id(face_.cplx().template nfaces<N>() - 1); } @@ -237,7 +237,7 @@ namespace mln template <unsigned N, unsigned D> inline void - complex_faces_bkd_iter_<N, D>::next_() + static_n_face_bkd_iter<N, D>::next_() { if (is_valid()) { @@ -264,4 +264,4 @@ namespace mln } // end of namespace mln -#endif // ! MLN_TOPO_COMPLEX_FACES_ITER_HH +#endif // ! MLN_TOPO_STATIC_N_FACE_ITER_HH diff --git a/milena/tests/topo/complex.cc b/milena/tests/topo/complex.cc index 3319813..4d220e7 100644 --- a/milena/tests/topo/complex.cc +++ b/milena/tests/topo/complex.cc @@ -35,9 +35,14 @@ using namespace mln; -// Forward declaration. -template <unsigned N, unsigned D> void test_faces_iter(topo::complex<D>& c); -template <unsigned N, unsigned D> void test_complex_faces_iter(topo::complex<D>& c); +// Forward declarations. +template <unsigned N, unsigned D> +void test_static_n_face_iter(topo::complex<D>& c); +// FIXME: Disabled (moved to the attic). +#if 0 +template <unsigned N, unsigned D> +void test_faces_iter(topo::complex<D>& c); +#endif int main() @@ -143,6 +148,11 @@ int main() actual complex processing since they are not really flexible --- but I'm not sure. */ + // Static version. + test_static_n_face_iter<0>(c); + test_static_n_face_iter<1>(c); + test_static_n_face_iter<2>(c); + // FIXME: Disabled (moved to the attic). # if 0 /* Using faces_{fwd,bkd}_iter_<N, D>, which are proxies to @@ -150,12 +160,6 @@ int main() test_faces_iter<0>(c); test_faces_iter<1>(c); test_faces_iter<2>(c); - - /* Using complex_faces_{fwd,bkd}_iter_<N, D>, which are proxies to - faces<N>. */ - test_complex_faces_iter<0>(c); - test_complex_faces_iter<1>(c); - test_complex_faces_iter<2>(c); #endif /*------------------------------. @@ -293,6 +297,22 @@ int main() } +template <unsigned N, unsigned D> +void +test_static_n_face_iter(topo::complex<D>& c) +{ + std::cout << "test_static_n_face_iter<" << N << ", " << D << ">:" + << std::endl; + /* FIXME: Provide sugar. Maybe redefined mln_fwd_fiter and + mln_bkd_fiter so that they expand as complex_faces_iters (instead + of faces_iters). */ + topo::static_n_face_fwd_iter<N, D> fwd_ncf(c); + topo::static_n_face_bkd_iter<N, D> bkd_ncf(c); + for_all_2(fwd_ncf, bkd_ncf) + std::cout << fwd_ncf << ' ' << bkd_ncf << std::endl; + std::cout << std::endl; +} + // FIXME: Disabled (moved to the attic). # if 0 template <unsigned N, unsigned D> @@ -307,20 +327,4 @@ test_faces_iter(topo::complex<D>& c) std::cout << fwd_nf << ' ' << bkd_nf << std::endl; std::cout << std::endl; } - -template <unsigned N, unsigned D> -void -test_complex_faces_iter(topo::complex<D>& c) -{ - std::cout << "test_complex_faces_iter<" << N << ", " << D << ">:" - << std::endl; - /* FIXME: Provide sugar. Maybe redefined mln_fwd_fiter and - mln_bkd_fiter so that they expand as complex_faces_iters (instead - of faces_iters). */ - topo::complex_faces_fwd_iter_<N, D> fwd_ncf(c); - topo::complex_faces_bkd_iter_<N, D> bkd_ncf(c); - for_all_2(fwd_ncf, bkd_ncf) - std::cout << fwd_ncf << ' ' << bkd_ncf << std::endl; - std::cout << std::endl; -} #endif -- 1.6.0.1

16 years, 3 months

2422: Adjust paths of included files in the attic.

by Roland Levillain

* mln/core/site_set/attic/p_complex_faces_piter.hh, * mln/core/site_set/attic/p_faces_piter.hh: Do it. --- milena/ChangeLog | 8 ++++++++ .../core/site_set/attic/p_complex_faces_piter.hh | 2 +- milena/mln/core/site_set/attic/p_faces_piter.hh | 2 +- 3 files changed, 10 insertions(+), 2 deletions(-) diff --git a/milena/ChangeLog b/milena/ChangeLog index 77d210c..d469c21 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,13 @@ 2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Adjust paths of included files in the attic. + + * mln/core/site_set/attic/p_complex_faces_piter.hh, + * mln/core/site_set/attic/p_faces_piter.hh: + Do it. + +2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Remove useless copy ctors and assignment operators in face handles. * mln/topo/face.hh, mln/topo/n_face.hh diff --git a/milena/mln/core/site_set/attic/p_complex_faces_piter.hh b/milena/mln/core/site_set/attic/p_complex_faces_piter.hh index de10d54..59ab7b4 100644 --- a/milena/mln/core/site_set/attic/p_complex_faces_piter.hh +++ b/milena/mln/core/site_set/attic/p_complex_faces_piter.hh @@ -37,7 +37,7 @@ # include <mln/core/site_set/p_complex.hh> # include <mln/core/internal/p_complex_piter_base.hh> -# include <mln/topo/complex_faces_iter.hh> +# include <mln/topo/attic/complex_faces_iter.hh> /* FIXME: p_complex_faces_fwd_piter_ and p_complex_faces_bkd_piter_ are really long names: find something shorter. */ diff --git a/milena/mln/core/site_set/attic/p_faces_piter.hh b/milena/mln/core/site_set/attic/p_faces_piter.hh index bfd1031..4f28082 100644 --- a/milena/mln/core/site_set/attic/p_faces_piter.hh +++ b/milena/mln/core/site_set/attic/p_faces_piter.hh @@ -36,7 +36,7 @@ # include <mln/core/site_set/p_faces.hh> # include <mln/core/internal/p_complex_piter_base.hh> -# include <mln/topo/faces_iter.hh> +# include <mln/topo/attic/faces_iter.hh> namespace mln -- 1.6.0.1

16 years, 3 months

2421: Remove useless copy ctors and assignment operators in face handles.

by Roland Levillain

* mln/topo/face.hh, mln/topo/n_face.hh (mln::face<D>::face(const face<D>&)) (mln::n_face<N, D>::n_face(const n_face<N, D>&)): Remove copy ctors. (mln::face<D>::operator=(const face<D>&)) (mln::n_face<N, D>::operator=(const n_face<N, D>&)): Remove assignment operators. --- milena/ChangeLog | 12 ++++++++++++ milena/mln/topo/face.hh | 28 ---------------------------- milena/mln/topo/n_face.hh | 29 ----------------------------- 3 files changed, 12 insertions(+), 57 deletions(-) diff --git a/milena/ChangeLog b/milena/ChangeLog index 6269489..77d210c 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,17 @@ 2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Remove useless copy ctors and assignment operators in face handles. + + * mln/topo/face.hh, mln/topo/n_face.hh + (mln::face<D>::face(const face<D>&)) + (mln::n_face<N, D>::n_face(const n_face<N, D>&)): + Remove copy ctors. + (mln::face<D>::operator=(const face<D>&)) + (mln::n_face<N, D>::operator=(const n_face<N, D>&)): + Remove assignment operators. + +2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Adjust complex piters w.r.t. new names of iterators on complex faces. * mln/core/site_set/p_complex_piter.hh: diff --git a/milena/mln/topo/face.hh b/milena/mln/topo/face.hh index 136271c..b35f6c7 100644 --- a/milena/mln/topo/face.hh +++ b/milena/mln/topo/face.hh @@ -70,13 +70,6 @@ namespace mln template <unsigned N> face(const n_face<N, D>& f); - // FIXME: Probably useless. - /// Copy and assignment. - /// \{ - face(const face<D>& rhs); - face<D>& operator=(const face<D>& rhs); - /// \} - /// Is this handle valid? bool is_valid() const; /// Invalidate this handle. @@ -176,27 +169,6 @@ namespace mln template <unsigned D> inline - face<D>::face(const face<D>& rhs) - : cplx_(rhs.cplx_), n_(rhs.n_), face_id_(rhs.face_id_) - { - } - - template <unsigned D> - inline - face<D>& - face<D>::operator=(const face<D>& rhs) - { - if (&rhs != this) - { - cplx_ = rhs.cplx_; - n_ = rhs.n_; - face_id_ = rhs.face_id_; - } - return *this; - } - - template <unsigned D> - inline bool face<D>::is_valid() const { diff --git a/milena/mln/topo/n_face.hh b/milena/mln/topo/n_face.hh index afeb4ab..e243cc6 100644 --- a/milena/mln/topo/n_face.hh +++ b/milena/mln/topo/n_face.hh @@ -66,13 +66,6 @@ namespace mln /// Build a face handle from \a complex and \a face_id. n_face(complex<D>& complex, unsigned face_id); - // FIXME: Probably useless. - /// Copy and assignment. - /// \{ - n_face(const n_face<N, D>& rhs); - n_face<N, D>& operator=(const n_face<N, D>& rhs); - /// \} - /// Is this handle valid? bool is_valid() const; /// Invalidate this handle. @@ -160,28 +153,6 @@ namespace mln template <unsigned N, unsigned D> inline - n_face<N, D>::n_face(const n_face<N, D>& rhs) - : cplx_(rhs.cplx_), face_id_(rhs.face_id_) - { - // Ensure N is compatible with D. - metal::bool_< N <= D >::check(); - } - - template <unsigned N, unsigned D> - inline - n_face<N, D>& - n_face<N, D>::operator=(const n_face<N, D>& rhs) - { - if (&rhs != this) - { - cplx_ = rhs.cplx_; - face_id_ = rhs.face_id_; - } - return *this; - } - - template <unsigned N, unsigned D> - inline bool n_face<N, D>::is_valid() const { -- 1.6.0.1

16 years, 3 months

2420: Adjust complex piters w.r.t. new names of iterators on complex faces.

by Roland Levillain

* mln/core/site_set/p_complex_piter.hh: s/topo::complex_fwd_iter_/topo::face_fwd_iter/ s/topo::complex_bkd_iter_/topo::face_bkd_iter/ Update included header's name. --- milena/ChangeLog | 9 +++++++++ milena/mln/core/site_set/p_complex_piter.hh | 10 +++++----- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/milena/ChangeLog b/milena/ChangeLog index e2d0609..6269489 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,14 @@ 2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Adjust complex piters w.r.t. new names of iterators on complex faces. + + * mln/core/site_set/p_complex_piter.hh: + s/topo::complex_fwd_iter_/topo::face_fwd_iter/ + s/topo::complex_bkd_iter_/topo::face_bkd_iter/ + Update included header's name. + +2008-09-26 Roland Levillain <roland(a)lrde.epita.fr> + Rename topo::complex_{fwd,bkd}_iter_ as topo::face_{fwd,bkd}_iter. * mln/topo/complex_iter.hh: Rename as... diff --git a/milena/mln/core/site_set/p_complex_piter.hh b/milena/mln/core/site_set/p_complex_piter.hh index 996b637..b9ec30e 100644 --- a/milena/mln/core/site_set/p_complex_piter.hh +++ b/milena/mln/core/site_set/p_complex_piter.hh @@ -36,7 +36,7 @@ # include <mln/core/site_set/p_complex.hh> # include <mln/core/internal/p_complex_piter_base.hh> -# include <mln/topo/complex_iter.hh> +# include <mln/topo/face_iter.hh> namespace mln { @@ -55,13 +55,13 @@ namespace mln /// \brief Forward iterator on (all) the faces of an mln::p_complex<D, P>. template <unsigned D, typename P> class p_complex_fwd_piter_ - : public internal::p_complex_piter_base_< topo::complex_fwd_iter_<D>, + : public internal::p_complex_piter_base_< topo::face_fwd_iter<D>, p_complex<D, P>, P, p_complex_fwd_piter_<D, P> > { typedef p_complex_fwd_piter_<D, P> self_; - typedef internal::p_complex_piter_base_< topo::complex_fwd_iter_<D>, + typedef internal::p_complex_piter_base_< topo::face_fwd_iter<D>, p_complex<D, P>, P, self_ > super_; @@ -82,13 +82,13 @@ namespace mln /// \brief Backward iterator on (all) the faces of an mln::p_complex<D, P>. template <unsigned D, typename P> class p_complex_bkd_piter_ - : public internal::p_complex_piter_base_< topo::complex_bkd_iter_<D>, + : public internal::p_complex_piter_base_< topo::face_bkd_iter<D>, p_complex<D, P>, P, p_complex_bkd_piter_<D, P> > { typedef p_complex_bkd_piter_<D, P> self_; - typedef internal::p_complex_piter_base_< topo::complex_bkd_iter_<D>, + typedef internal::p_complex_piter_base_< topo::face_bkd_iter<D>, p_complex<D, P>, P, self_ > super_; -- 1.6.0.1

16 years, 3 months

← Newer
1
...
4
5
6
7
8
9
10
...
36
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Olena-patches September 2008