rename Task::erase -> eraseTaskCallback

`erase` is too general for a container class,
when it's not actually about erasing children or the container itself.

Author: v4hn

core/include/moveit/task_constructor/task.h

@@ -105,7 +105,7 @@ class Task : protected WrapperBase
 	/// add function to be called after each top-level iteration
 	TaskCallbackList::const_iterator addTaskCallback(TaskCallback&& cb);
 	/// remove function callback
-	void erase(TaskCallbackList::const_iterator which);
+	void eraseTaskCallback(TaskCallbackList::const_iterator which);
 
 	/// expose SolutionCallback API
 	using WrapperBase::SolutionCallback;

core/src/task.cpp

@@ -235,7 +235,7 @@ Task::TaskCallbackList::const_iterator Task::addTaskCallback(TaskCallback&& cb)
 	return --(impl->task_cbs_.cend());
 }
 
-void Task::erase(TaskCallbackList::const_iterator which) {
+void Task::eraseTaskCallback(TaskCallbackList::const_iterator which) {
 	pimpl()->task_cbs_.erase(which);
 }