cleanup: use singleton pattern for getPrintingTemplatePath*()

Function-local statics are initialized on first invocation.
No point in extra logic.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
This commit is contained in:
Berthold Stoeger 2022-09-24 19:42:07 +02:00
parent f407f5269a
commit 727d519046

View file

@ -719,17 +719,17 @@ static const char *printing_templates = "printing_templates";
QString getPrintingTemplatePathUser() QString getPrintingTemplatePathUser()
{ {
static QString path = QString(); // Function-local statics are initialized on first invocation
if (path.isEmpty()) static QString path(QString(system_default_directory()) +
path = QString(system_default_directory()) + QDir::separator() + QString(printing_templates); QDir::separator() +
QString(printing_templates));
return path; return path;
} }
QString getPrintingTemplatePathBundle() QString getPrintingTemplatePathBundle()
{ {
static QString path = QString(); // Function-local statics are initialized on first invocation
if (path.isEmpty()) static QString path(getSubsurfaceDataPath(printing_templates));
path = getSubsurfaceDataPath(printing_templates);
return path; return path;
} }