Every file in this commit makes a call to a function which returns a value.

Each file also didn't check the return value.

This commit changes it so that every single non-checked call in these files
is checked.

1 files changed, 8 insertions, 4 deletions

diff --git a/noncore/todayplugins/stockticker/stockticker/helpwindow.cpp b/noncore/todayplugins/stockticker/stockticker/helpwindow.cpp
index 410d642..2498bf9 100644
--- a/noncore/todayplugins/stockticker/stockticker/helpwindow.cpp
+++ b/noncore/todayplugins/stockticker/stockticker/helpwindow.cpp
@@ -182,10 +182,11 @@ HelpWindow::~HelpWindow()
   history.append( *it );
 
     QFile f( QDir::currentDirPath() + "/.history" );
-    f.open( IO_WriteOnly );
+    if ( f.open( IO_WriteOnly ) ) {
     QDataStream s( &f );
     s << history;
     f.close();
+    }
 
     bookmarks.clear();
     QMap<int, QString>::Iterator it2 = mBookmarks.begin();
@@ -193,7 +194,8 @@ HelpWindow::~HelpWindow()
   bookmarks.append( *it2 );
 
     QFile f2( QDir::currentDirPath() + "/.bookmarks" );
-    f2.open( IO_WriteOnly );
+    if ( !f2.open( IO_WriteOnly ) )
+        return;
     QDataStream s2( &f2 );
     s2 << bookmarks;
     f2.close();
@@ -232,7 +234,8 @@ void HelpWindow::readHistory()
 {
     if ( QFile::exists( QDir::currentDirPath() + "/.history" ) ) {
   QFile f( QDir::currentDirPath() + "/.history" );
-  f.open( IO_ReadOnly );
+        if ( !f.open( IO_ReadOnly ) )
+            return;
   QDataStream s( &f );
   s >> history;
   f.close();
@@ -245,7 +248,8 @@ void HelpWindow::readBookmarks()
 {
     if ( QFile::exists( QDir::currentDirPath() + "/.bookmarks" ) ) {
   QFile f( QDir::currentDirPath() + "/.bookmarks" );
-  f.open( IO_ReadOnly );
+            if ( !f.open( IO_ReadOnly ) )
+                return;
   QDataStream s( &f );
   s >> bookmarks;
   f.close();