From 1c81a600ebbc4ec97bccb6af4c97549e4d336d91 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Mikko=20Kein=C3=A4nen?= Date: Sat, 19 Jun 2010 18:40:00 +0300 Subject: [PATCH] comment --- src/utils/fileutil.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/utils/fileutil.cpp b/src/utils/fileutil.cpp index 410d0b8..a4ded34 100644 --- a/src/utils/fileutil.cpp +++ b/src/utils/fileutil.cpp @@ -40,7 +40,12 @@ QList FileUtil::scanFilePath(const FilePathObject *fp, QSt "" /* TODO */, fileInfo.size(), files, - fp->getSetup() // TODO: maybe assign a copy of the setup object, we cannot guarantee that the setup object in file path object will stay alive!!! + // TODO: is it guaranteed, that the file path object containing the setup object remains alive + // the whole lifecycle of (this) media image container object? + // * if we assign a copy of the setup object -> waste of memory and time + // * this function is designed to be used from media image path main dialog + // where we can ensure the lifecycle of file path object -> maybe move the implementation there!? + fp->getSetup() ); containers.append(con); } -- 1.7.9.5