[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdeartwork/kscreensaver/kdesavers
From:       Chris Howells <howells () kde ! org>
Date:       2004-08-26 17:33:55
Message-ID: 20040826173355.4209C90B5 () office ! kde ! org
[Download RAW message or body]

CVS commit by howells: 

This is a partial fix for 76746. It at least stops the screensaver from crashing if a \
non existing  directory is added.

the screensaver crashes on the QFileInfoListIterator it(*fileinfolist); line but \
unfortunately my Qt is  compiled without debugging symbols and my poor iBook 800 \
really can't deal with recompiling the whole lot  with debugging symbols again so I'm \
going to leave this bug open.

Also I'm not sure whether this should be considered a bug in KUrlRequester or not -- \
there is a  KFile::LocalOnly mask, but this only stops local files being shown in the \
dialog, maybe KUrlRequester  should have an isLocal() function or similar if a lan:// \
or something URL is entered.

CCMAIL: 76746@bugs.kde.org

----------------------------------------------------------------------


  M +4 -0      slideshow.cpp   1.25


--- kdeartwork/kscreensaver/kdesavers/slideshow.cpp  #1.24:1.25
@@ -865,4 +865,8 @@ void kSlideShowSaver::traverseDirectory(
 {
   QDir dir(dirName);
+  if (!dir.exists())
+  {
+     return ;
+  }
   dir.setFilter(QDir::Dirs | QDir::Files);
 


[prev in list] [next in list] [prev in thread] [next in thread]