The referred comment is not in the recent versions anymore, but the same way is still done.
Here is the old code referred (v0.8.3):
[...]
// transform new words in local map referential
//inliers2 = util3d::transformPointCloud<pcl::PointXYZ>(inliers2, this->getPose());
// the transform returned is global odometry pose, not incremental one
std::vector<int> inliersV;
transform = util3d::transformFromXYZCorrespondences(
inliers2,
inliers1,
this->getInlierDistance(),
this->getIterations(),
this->getRefineIterations()>0, 3.0, this->getRefineIterations(),
&inliersV,
&variance);
inliers = (int)inliersV.size();
if(!transform.isNull())
{
// make it incremental
transform = this->getPose().inverse() * transform;
[...]
We don't have to transform the inliers2 in the odom frame, the returned transform returns the actual pose of the camera in odom frame. To make the transform incremental (transform between the last and the new camera pose), this is done on this line:
transform = this->getPose().inverse() * transform;
For example, if we transform the inliers2 cloud in odom frame before computing the transform, we have directly the transform between old pose and new pose (don't need to make it incremental). The computation cost of the first method imply only a transform multiplication and avoid the prior transformation of the all points of inliers2.