Switch to using scoped_ptr for bbox_collision_detector impl
Switch to using scoped_ptr for bbox_collision_detector impl

I think this fixes no memory leaks.  But now it's guaranteed!
Scoped_ptr is watertight as long as you put "new"s into it
directly in one statement that contains nothing else
(i.e. in scoped_ptr's constructor or .reset()).

Lasercake/Lasercake,elidupree/Lasercake,Lasercake/Lasercake,elidupree/Lasercake,elidupree/Lasercake,elidupree/Lasercake,idupree/Lasercake,idupree/Lasercake,Lasercake/Lasercake,idupree/Lasercake,elidupree/Lasercake,Lasercake/Lasercake,Lasercake/Lasercake,idupree/Lasercake,idupree/Lasercake,elidupree/Lasercake,idupree/Lasercake,elidupree/Lasercake,idupree/Lasercake