xf86-video-intel: src/sna/sna_display.c

Chris Wilson ickle at kemper.freedesktop.org
Sat Feb 21 09:09:16 PST 2015


 src/sna/sna_display.c |    3 +++
 1 file changed, 3 insertions(+)

New commits:
commit fef0f3dc0ca8d7f6e50fd841a7fff23a4d357121
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Sat Feb 21 17:08:06 2015 +0000

    sna: Add missing 'break' from last patch
    
    Converting from
    
    switch(value)
    {
    case 1: return 1;
    ...
    default: return -1;
    }
    
    you must remember to insert the breaks!
    
    Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

diff --git a/src/sna/sna_display.c b/src/sna/sna_display.c
index 657e998..606c13a 100644
--- a/src/sna/sna_display.c
+++ b/src/sna/sna_display.c
@@ -2968,11 +2968,14 @@ sna_output_detect(xf86OutputPtr output)
 	switch (compat_conn.conn.connection) {
 	case DRM_MODE_CONNECTED:
 		sna_output->status = XF86OutputStatusConnected;
+		break;
 	case DRM_MODE_DISCONNECTED:
 		sna_output->status = XF86OutputStatusDisconnected;
+		break;
 	default:
 	case DRM_MODE_UNKNOWNCONNECTION:
 		sna_output->status = XF86OutputStatusUnknown;
+		break;
 	}
 	return sna_output->status;
 }


More information about the xorg-commit mailing list