Bug 203130

Summary: assert(false) in Viewer/ImageDisplay.cpp
Product: [Applications] kphotoalbum Reporter: Jan Kundrát <jkt>
Component: ViewerAssignee: KPhotoAlbum Bugs <kpabugs>
Status: RESOLVED WORKSFORME    
Severity: normal CC: miika.turkia
Priority: NOR    
Version: SVN trunk (KDE4 version)   
Target Milestone: ---   
Platform: Unlisted Binaries   
OS: Linux   
Latest Commit: Version Fixed In:

Description Jan Kundrát 2009-08-09 03:29:00 UTC
diff --git a/devel/Viewer/ImageDisplay.cpp b/devel/Viewer/ImageDisplay.cpp
index 5efeb74..512c088 100644
--- a/devel/Viewer/ImageDisplay.cpp
+++ b/devel/Viewer/ImageDisplay.cpp
@@ -461,6 +461,11 @@ void Viewer::ImageDisplay::updatePreload()
                 }

                 Q_ASSERT( false ); // We should never get here.
+                // FIXME: we do end here whenever the "cache size" is set too low.
+                // How to reproduce:
+                // a) Set cache size to 1 MB
+                // b) View an image
+                // c) Press PgDn
             }

             return;
Comment 1 Miika Turkia 2010-01-02 09:14:28 UTC
Is this the same bug I discovered also and fixed? (My fix was for cache size 0, but at first try I could not reproduce this bug with size 1. Well, with cache size 1 the official release of kphotoalbum 4.1.1 did not crash for me either.)

Revision 1064366 - (view) (download) (annotate) - [select for diffs]
Modified Sun Dec 20 18:57:13 2009 UTC (12 days, 13 hours ago) by mturkia

bugfix: KPA crashes when viewing images if viewer cache size is set to zero