Bug 412853 many strict warnings [anonymous function does not always return a value]; r=philipp

calendar/base/content/calendar-task-tree.xml

@@ -285,7 +285,8 @@
                   // need to act accordingly.
                   if (this.binding.mFilterFunction &&
                       !this.binding.mFilterFunction(aNewItem)) {
-                      return this.removeItem(aNewItem);
+                      this.removeItem(aNewItem);
+                      return;
                   }
                   // same holds true for the completed filter, which is
                   // currently modeled as an explicit boolean.

calendar/base/src/calCalendarManager.js

@@ -471,6 +471,7 @@ calCalendarManager.prototype = {
                                 "_blank",
                                 "chrome,dialog=yes,alwaysRaised=yes",
                                 paramBlock);
+            return null;
         }
     },
 

calendar/providers/memory/calMemoryCalendar.js

@@ -255,6 +255,7 @@ calMemoryCalendar.prototype = {
                                            modifiedItem);
         // notify observers
         this.mObservers.notify("onModifyItem", [modifiedItem, aOldItem]);
+        return null;
     },
 
     // void deleteItem( in calIItemBase aItem, in calIOperationListener aListener );

calendar/providers/storage/calStorageCalendar.js

@@ -422,7 +422,7 @@ calStorageCalendar.prototype = {
                                               null,
                                               "Calendar is readonly");
             }
-            return;
+            return null;
         }
         if (!aNewItem) {
             throw Components.results.NS_ERROR_INVALID_ARG;
@@ -486,6 +486,7 @@ calStorageCalendar.prototype = {
 
         // notify observers
         this.mObservers.notify("onModifyItem", [modifiedItem, aOldItem]);
+        return null;
     },
 
     // void deleteItem( in string id, in calIOperationListener aListener );