Don't ignore IOException stupid!

Files may not be accessible if they are read-only, or if the directory
is read-only, or ... whatever. So, let the caller handle IOException
in DataWriter.

Return a set if it can be read.

Do some misc cleanups (remove unused imports, fix docs, remove obsolete
TODOs).

1 files changed, 10 insertions, 5 deletions

diff --git a/src/main/TweetShell.java b/src/main/TweetShell.java
index 3ab1397..a5114e2 100644
--- a/src/main/TweetShell.java
+++ b/src/main/TweetShell.java
@@ -377,11 +377,16 @@ public class TweetShell implements TwitterApi.PinSupplier {
 
             String profilesFilename = config.getProperty(DataWriter.CFG_PROFILE_FILENAME);
             String tweetsFilename = config.getProperty(DataWriter.CFG_TWEETS_FILENAME);
-
-            resultListeners.register(new DataWriter(profilesFilename, tweetsFilename));
-
-            // save the changes to the config.
-            config.save();
+            try {
+                DataWriter dw = new DataWriter(profilesFilename, tweetsFilename);
+                resultListeners.register(dw);
+                // save the changes to the config.
+                config.save();
+            } catch (IOException ex) {
+                System.err.println("Could not open file for storing tweets:");
+                System.err.println(ex.getMessage());
+                return false;
+            }
         } else if (rlCls == StreamHandler.class) {
             resultListeners.register(new StreamHandler());
         }