Bug 777292 - Add default cases to switches on nsresult; r=ehsan

Brli · Jul 27, 2012 · 6e58958 · 6e58958
1 parent 22f3c50
commit 6e58958
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/docshell/base/nsDocShell.cpp b/docshell/base/nsDocShell.cpp
@@ -4140,6 +4140,8 @@ nsDocShell::DisplayLoadError(nsresult aError, nsIURI *aURI,
             // Broken Content Detected. e.g. Content-MD5 check failure.
             error.AssignLiteral("corruptedContentError");
             break;
+        default:
+            break;
         }
     }
 

diff --git a/js/xpconnect/src/XPCWrappedJSClass.cpp b/js/xpconnect/src/XPCWrappedJSClass.cpp
@@ -101,9 +101,9 @@ JSBool xpc_IsReportableErrorCode(nsresult code)
         case NS_ERROR_FACTORY_REGISTER_AGAIN:
         case NS_BASE_STREAM_WOULD_BLOCK:
             return false;
+        default:
+            return true;
     }
-
-    return true;
 }
 
 // static
-Original file line number
+Diff line change
@@ Expand Up @@
                 // Broken Content Detected. e.g. Content-MD5 check failure.
                 error.AssignLiteral("corruptedContentError");
                 break;
+            default:
+                break;
             }
         }
@@ Expand Down @@