[PATCH] jscript: Mark up fallthrough switch cases (Coverity)

Marcus Meissner meissner at suse.de
Thu Mar 17 19:50:23 CDT 2011


Hi,

I am quite sure they should fall through, so mark them
up as such.

Ciao, Marcus
---
 dlls/jscript/jscript.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/dlls/jscript/jscript.c b/dlls/jscript/jscript.c
index a69b293..641c192 100644
--- a/dlls/jscript/jscript.c
+++ b/dlls/jscript/jscript.c
@@ -164,7 +164,7 @@ static void decrease_state(JScript *This, SCRIPTSTATE state)
             change_state(This, SCRIPTSTATE_DISCONNECTED);
             if(state == SCRIPTSTATE_DISCONNECTED)
                 return;
-
+            /* FALLTHROUGH */
         case SCRIPTSTATE_STARTED:
         case SCRIPTSTATE_DISCONNECTED:
             clear_script_queue(This);
@@ -173,7 +173,7 @@ static void decrease_state(JScript *This, SCRIPTSTATE state)
                 change_state(This, SCRIPTSTATE_INITIALIZED);
             if(state == SCRIPTSTATE_INITIALIZED)
                 return;
-
+            /* FALLTHROUGH */
         case SCRIPTSTATE_INITIALIZED:
             if(This->ctx->host_global) {
                 IDispatch_Release(This->ctx->host_global);
@@ -211,7 +211,7 @@ static void decrease_state(JScript *This, SCRIPTSTATE state)
                 jsdisp_release(This->ctx->global);
                 This->ctx->global = NULL;
             }
-
+            /* FALLTHROUGH */
         case SCRIPTSTATE_UNINITIALIZED:
             change_state(This, state);
             break;
-- 
1.7.1



More information about the wine-patches mailing list