Yes, using an existing implementation is a good idea, but why not just use std::abs?