Matthieu Garrigues <garrigues@lrde.epita.fr> writes:
URL: https://svn.lrde.epita.fr/svn/oln/trunk/milena
ChangeLog: 2007-10-26 Matthieu Garrigues <garrigues@lrde.epita.fr>
Fix warnings in some tests. * tests/border_resize_sub_image.cc, * tests/branch_iter.cc: Fix.
Be more explicit, ``Fix'' is not enough. [...]
Index: trunk/milena/tests/border_resize_sub_image.cc =================================================================== --- trunk/milena/tests/border_resize_sub_image.cc (revision 1395) +++ trunk/milena/tests/border_resize_sub_image.cc (revision 1396) @@ -60,7 +60,7 @@
typedef image2d<int> I; unsigned border = 42; - unsigned new_border = 51; + //unsigned new_border = 51;
No dead code please; or flag it with a FIXME.
Index: trunk/milena/tests/branch_iter.cc =================================================================== --- trunk/milena/tests/branch_iter.cc (revision 1395) +++ trunk/milena/tests/branch_iter.cc (revision 1396) @@ -47,7 +47,7 @@ util::node<int>* f = n.add_child(42);
util::node<int>* g = f->add_child(421); - util::node<int>* h = f->add_child(422); + f->add_child(422);
g->add_child(4211)->add_child(51)->add_child(52)->add_child(53)->add_child(54)->add_child(55);
This line is too long. You can rewrite is as: g-> add_child(4211)-> add_child(51)-> add_child(52)-> add_child(53)-> add_child(54)-> add_child(55);