yade-dev team mailing list archive
-
yade-dev team
-
Mailing list archive
-
Message #05254
Re: [Branch ~yade-dev/yade/trunk] Rev 2344: - miniEigen : memory leak source suspected (FIXME added)
-
To:
yade-dev@xxxxxxxxxxxxxxxxxxx
-
From:
Janek Kozicki <janek_listy@xxxxx>
-
Date:
Mon, 12 Jul 2010 12:59:53 +0200
-
Face:
iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAALVBMVEUBAQEtLS1KSkpRUVFXV1dYWFhjY2Nzc3N3d3eHh4eKioqdnZ24uLjLy8vc3NxVIagyAAAACXBIWXMAAAsTAAALEwEAmpwYAAAAB3RJTUUH2AIVEzgS1fgQtQAAAjRJREFUOMtt1DFv00AUAOAzFQNbjigSyoQaRaBMhKgLUyKXpVNNeUpk9vyDqFJhQ1kiBuaqAwJCqvPtSLY7RlTn5+5IdnYkkt/AOyfxXVLe5vf53Z1875kd34tOEax8djmj6GyjhB5bxz50GdsVZr9fqRjZwAtKOJw5Wqs2MMZ16ALHsaDncF7xAHix1oEFHAB8f+pRjcO4gfZDykcYzbiucRolOLUJ6kjA0xtVt+A6TySlM0RajIpK6DzwKZ/nOYbF/gclHMo1ZOHYY/+Ha+AWuM+3oMS4eeqYzZ8FiCltgUqI8cd2wwAVpJk+8LWYjBtnJdQpHQqJMd4Oxt4bU9ESiFGc5hkqaH74asAX4iabP5I5gZ+qjgGlJCqZa3h3lxhoeVcSE1qLQC4sqKOK9MGW9E3izFqqHokoztLFEgXg31sbZEKnWi2T74A4NxfVQqlkjKtcAWD+zcArFEES01dR0E/nnV0IgugmDd/2L84sOAouRBBHEc7gtc8teDkRlE0iNQPo2w3Xhh/D4TCIQ4LRLoTvgwjj6RRgavdurxYGMaIuGOyAW/PpNlCcU9/93AHenAWYjPoAwa+G3e3to/MgFNTAEKvKDjzuCzHTnY3qqdXtx24VijzQfZ0yewZ5cwRFQaa+mIYr1uI0I76+3W4xhlvoVRwOA0Fdl64HlJnxP6T8YpX/Lga4Wv4A3ErrU5oTfN7Mu/llXMl8RXEPji/lQkN3H7qXqgC2By47EXeU/7PJ/wPxRKMnuZwIeAAAAABJRU5ErkJggg==
-
In-reply-to:
<AANLkTiknmWvFIvT4TNtoBeM20Ffb78o8A_9eKaKE2DfR@mail.gmail.com>
Anton Gladky said: (by the date of Mon, 12 Jul 2010 12:38:04 +0200)
> >
> > use bzr blame to find who wrote this. This code is crazy wrong.
> >
>
> Can you propose something better?
why it must return a pointer? couldn't be just like that:
Quaternionr Quaternionr_fromAxisAngle(const Vector3r& axis, const Real angle){ return Quaternionr(AngleAxisr(angle,axis)); }
Quaternionr Quaternionr_fromAngleAxis(const Real angle, const Vector3r& axis){ return Quaternionr(AngleAxisr(angle,axis)); }
but.. umm this code isn't use anywhere, or I am blind?
trunk$ grep Quaternionr_fromAxisAngle ./* -Rl
./py/mathWrap/miniEigen.cpp
trunk$ grep Quaternionr_fromAngleAxis ./* -Rl
./py/mathWrap/miniEigen.cpp
oh, I'm sorry... it's used for a python wrapper line 263
.def("__init__",bp::make_constructor(&Quaternionr_fromAxisAngle,bp::default_call_policies(),(bp::arg("axis"),bp::arg("angle"))))
.def("__init__",bp::make_constructor(&Quaternionr_fromAngleAxis,bp::default_call_policies(),(bp::arg("angle"),bp::arg("axis"))))
why python needs this to be a pointer?
--
Janek Kozicki http://janek.kozicki.pl/ |
Follow ups
References