Index: olena/oln/morpho/geodesic_dilation.hh --- olena/oln/morpho/geodesic_dilation.hh Sun, 14 Mar 2004 18:21:09 +0100 palma_g (oln/37_geodesic_d 1.11.1.12 600) +++ olena/oln/morpho/geodesic_dilation.hh Tue, 13 Apr 2004 23:03:36 +0200 thivol_d (oln/37_geodesic_d 1.11.1.12 600) @@ -69,7 +69,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** save(oln::morpho::geodesic_dilation(im2, im1, oln::neighb_c4()), ** IMG_OUT "oln_morpho_geodesic_dilation.pbm"); @@ -130,7 +130,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** save(oln::morpho::sure::geodesic_dilation(im2, im1, oln::neighb_c4()), ** IMG_OUT "oln_morpho_sure_geodesic_dilation.pbm"); Index: olena/oln/morpho/geodesic_erosion.hh --- olena/oln/morpho/geodesic_erosion.hh Sun, 14 Mar 2004 18:21:09 +0100 palma_g (oln/36_geodesic_e 1.19 600) +++ olena/oln/morpho/geodesic_erosion.hh Tue, 13 Apr 2004 23:03:01 +0200 thivol_d (oln/36_geodesic_e 1.19 600) @@ -67,7 +67,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** save(oln::morpho::geodesic_erosion(im1, im2, oln::neighb_c4()), ** IMG_OUT "oln_morpho_geodesic_erosion.pbm"); @@ -128,7 +128,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** save(oln::morpho::sure::geodesic_erosion(im1, im2, oln::neighb_c4()), ** IMG_OUT "oln_morpho_sure_geodesic_erosion.pbm"); Index: olena/oln/morpho/reconstruction.hh --- olena/oln/morpho/reconstruction.hh Sun, 14 Mar 2004 18:21:09 +0100 palma_g (oln/35_reconstruc 1.13.1.13 600) +++ olena/oln/morpho/reconstruction.hh Tue, 13 Apr 2004 23:02:25 +0200 thivol_d (oln/35_reconstruc 1.13.1.13 600) @@ -74,7 +74,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** oln::save(oln::morpho::sure::geodesic_reconstruction_dilation(im2, ** im1, @@ -146,7 +146,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** oln::save(oln::morpho::sequential::geodesic_reconstruction_dilation(im2, ** im1, @@ -262,7 +262,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** oln::save(oln::morpho::hybrid::geodesic_reconstruction_dilation(im2, ** im1, @@ -367,7 +367,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** oln::save(oln::morpho::sure::geodesic_reconstruction_erosion(im1, ** im2, @@ -440,7 +440,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** oln::save(oln::morpho::sequential::geodesic_reconstruction_erosion(im1, ** im2, @@ -552,7 +552,7 @@ ** typedef oln::image2dntg::int_u8 im_type; ** ** im_type im1(oln::load(IMG_IN "lena128.pgm")); - ** im_type im2(oln::morpho::opening(im1, oln::win_c4p())); + ** im_type im2 = oln::morpho::opening(im1, oln::win_c4p()); ** ** oln::save(oln::morpho::hybrid::geodesic_reconstruction_erosion(im1, ** im2, Index: olena/oln/topo/tarjan/flat-zone.hh --- olena/oln/topo/tarjan/flat-zone.hh Fri, 02 Apr 2004 16:53:54 +0200 palma_g (oln/r/35_flat-zone. 1.13 600) +++ olena/oln/topo/tarjan/flat-zone.hh Wed, 14 Apr 2004 00:03:57 +0200 thivol_d (oln/r/35_flat-zone. 1.13 600) @@ -88,7 +88,7 @@ typedef abstract::tarjan_with_attr<exact_type> super_type; ///< Type of parent class.
friend class super_type; - friend class super_type::super_type; + friend class abstract::tarjan<exact_type>; /*! ** \brief Constructor. **
"Damien" == Damien Thivolle damien@lrde.epita.fr writes:
Ce patch est sans ChangeLog, ni justification : *pourquoi* ICC demande ce changement ???
Akim Demaille wrote:
"Damien" == Damien Thivolle damien@lrde.epita.fr writes:
Ce patch est sans ChangeLog,
Index: olena/ChangeLog from Damien Thivolle damien@lrde.epita.fr
* olena/oln/morpho/geodesic_dilation.hh: Fix to make compile with ICC. * olena/oln/morpho/geodesic_erosion.hh: Likewise. * olena/oln/morpho/reconstruction.hh: Likewise. * olena/oln/topo/tarjan/flat-zone.hh: Likewise.
ni justification : *pourquoi* ICC demande ce changement ???
1) img_type out (algo(in)); 2) img_type out = algo(in);
L'écriture (1) ne fonctionne pas pour certains algos. L'écriture (2) fonctionne toujours et doit toujours étre utilisée.
La raison est certainement compliquée. Le problème vient du constructeur par copie des images qui n'est pas implémenté, mais qui est appelé par ICC certainement à cause de problèmes de constitude.
Nous avons cherché comment réparer et non pourquoi c'est cassé. J'attends d'avoir le temps de comprendre le pourquoi.
"Niels" == Niels Van Vliet niels@lrde.epita.fr writes:
Akim Demaille wrote:
"Damien" == Damien Thivolle damien@lrde.epita.fr writes:
Ce patch est sans ChangeLog,
Index: olena/ChangeLog from Damien Thivolle damien@lrde.epita.fr
- olena/oln/morpho/geodesic_dilation.hh: Fix to make compile with ICC.
- olena/oln/morpho/geodesic_erosion.hh: Likewise.
- olena/oln/morpho/reconstruction.hh: Likewise.
- olena/oln/topo/tarjan/flat-zone.hh: Likewise.
ni justification : *pourquoi* ICC demande ce changement ???
- img_type out (algo(in));
- img_type out = algo(in);
L'écriture (1) ne fonctionne pas pour certains algos. L'écriture (2) fonctionne toujours et doit toujours étre utilisée.
La raison est certainement compliquée. Le problème vient du constructeur par copie des images qui n'est pas implémenté, mais qui est appelé par ICC certainement à cause de problèmes de constitude.
Nous avons cherché comment réparer et non pourquoi c'est cassé. J'attends d'avoir le temps de comprendre le pourquoi.
OK.
Est-ce bien noté qqpart ?
Akim Demaille wrote:
"Niels" == Niels Van Vliet niels@lrde.epita.fr writes: [...]
- img_type out (algo(in));
- img_type out = algo(in);
L'écriture (1) ne fonctionne pas pour certains algos. L'écriture (2) fonctionne toujours et doit toujours étre utilisée. [...]
OK.
Est-ce bien noté qqpart ?
c'est dû aux bizarreries qu'on doit écrire pour gérer la constance des images ; les détails sont lisibles ici :
http://www.lrde.epita.fr/cgi-bin/twiki/view/Projects/OlenaConstImages
"Théo" == Thierry Géraud thierry.geraud@lrde.epita.fr writes:
Akim Demaille wrote:
"Niels" == Niels Van Vliet niels@lrde.epita.fr writes: [...]
- img_type out (algo(in));
- img_type out = algo(in);
L'écriture (1) ne fonctionne pas pour certains algos. L'écriture (2) fonctionne toujours et doit toujours étre utilisée. [...]
OK. Est-ce bien noté qqpart ?
c'est dû aux bizarreries qu'on doit écrire pour gérer la constance des images ; les détails sont lisibles ici :
http://www.lrde.epita.fr/cgi-bin/twiki/view/Projects/OlenaConstImages
Alors le ChangeLog devrait clairement pointer dessus !
-
Akim Demaille -
Damien Thivolle -
Niels Van Vliet -
Thierry Géraud