tasn pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=f44d786fa1efccb2ede5b7a11c3d72bb005d87ba

commit f44d786fa1efccb2ede5b7a11c3d72bb005d87ba
Author: Tom Hacohen <t...@stosb.com>
Date:   Tue Dec 8 11:05:17 2015 +0000

    Ecore con socks: Add missing break statement to switch.
    
    This looks like an obvious case of missing break. If it wasn't a missing
    break, there should have been at least a comment. Looking at the code it
    looks like a break is needed. Also, I suspect this code path is never
    really tested, and that's why we never hit it.
    
    Tests are not failing either way.
    
    CID1039379
---
 src/lib/ecore_con/ecore_con_socks.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/lib/ecore_con/ecore_con_socks.c 
b/src/lib/ecore_con/ecore_con_socks.c
index e6226e3..a017f2b 100644
--- a/src/lib/ecore_con/ecore_con_socks.c
+++ b/src/lib/ecore_con/ecore_con_socks.c
@@ -382,6 +382,7 @@ _ecore_con_socks_read_v5(Ecore_Con_Server *obj, 
Ecore_Con_Socks_v5 *v5, const un
            default:
              break;
           }
+        break;
 
       case ECORE_CON_PROXY_STATE_REQUEST:
       {

-- 


Reply via email to