[Olena-patches] last-svn-commit-968-g1eb0e9a Add comments.

* mln/accu/take.hh, * mln/border/mirror.hh: Here. --- milena/ChangeLog | 7 +++++++ milena/mln/accu/take.hh | 2 ++ milena/mln/border/mirror.hh | 10 ++++++++-- 3 files changed, 17 insertions(+), 2 deletions(-) diff --git a/milena/ChangeLog b/milena/ChangeLog index 6dd4556..2902026 100644 --- a/milena/ChangeLog +++ b/milena/ChangeLog @@ -1,5 +1,12 @@ 2011-09-07 Guillaume Lazzara &lt;z(a)lrde.epita.fr&gt; + Add comments. + + * mln/accu/take.hh, + * mln/border/mirror.hh: Here. + +2011-09-07 Guillaume Lazzara &lt;z(a)lrde.epita.fr&gt; + * tests/border/mirror.cc: Revamp test. 2011-09-06 Guillaume Lazzara &lt;z(a)lrde.epita.fr&gt; diff --git a/milena/mln/accu/take.hh b/milena/mln/accu/take.hh index 7bafbac..56bdaba 100644 --- a/milena/mln/accu/take.hh +++ b/milena/mln/accu/take.hh @@ -34,6 +34,8 @@ # include <mln/core/concept/image.hh> # include <mln/util/pix.hh> +// FIXME: to be removed? Seems to duplicate data::compute and +// set::compute. Moreover it does not work... namespace mln { diff --git a/milena/mln/border/mirror.hh b/milena/mln/border/mirror.hh index 183c0a2..793ef21 100644 --- a/milena/mln/border/mirror.hh +++ b/milena/mln/border/mirror.hh @@ -1,4 +1,5 @@ -// Copyright (C) 2007, 2008, 2009 EPITA Research and Development Laboratory (LRDE) +// Copyright (C) 2007, 2008, 2009, 2011 EPITA Research and Development +// Laboratory (LRDE) // // This file is part of Olena. // @@ -31,7 +32,10 @@ /// Define function that fills border using nearer pixels with a /// mirroring effect. /// -/// \todo Awful code: rewrite it! +/// \fixme 2D version is not correct if the border is larger than the +/// image domain. +/// +/// \todo Awful code: rewrite it! # include <mln/core/image/image1d.hh> # include <mln/core/image/image2d.hh> @@ -193,6 +197,8 @@ namespace mln inline void mirror_(const box3d&, const I& ima) { + trace::warning("border::mirror for 3D image is not implemented," + " so image borders have not been mirrored!"); mln::internal::fixme(); } -- 1.7.2.5