since we're unconditionally allocating this list, just declare it
inline.
And make it a std::vector instead of a std::list, it very inefficient
storing pointers in a linked list

Change-Id: I715868265f9f2e633395f474267e84ee87356726
Reviewed-on: https://gerrit.libreoffice.org/51317Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>