Make FontInfo::scan return a std::vector object

... rather than a pointer to a std::vector.  Given that a
std::vector is little more than a pointer and some size
information, there is no need to create std::vector objects
on the heap.  Returning them by value is just as fast
(the vector content is not copied), and makes the code
more readable, too.