Hi Keith, This regressed compilation of the Intel driver in gnome Continuous.
I've attached a patch which fixes it here.
From 3bfd4178341f8eb2c22ff3287478ce3c8a542028 Mon Sep 17 00:00:00 2001 From: Colin Walters <[email protected]> Date: Fri, 18 Jul 2014 08:11:14 -0400 Subject: [PATCH] xf86platformBus: Add assertion to avoid (fatal) compiler warning Compilation of -video-intel started failing in gnome-continuous, it's because xserver has -Werror=return-type on, and gcc can't prove this function always returns a value: /usr/include/xorg/xf86platformBus.h:119:1: error: control reaches end of non-void function [-Werror=return-type] Let's add assertions to the accessor functions to fix this. --- hw/xfree86/common/xf86platformBus.h | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/hw/xfree86/common/xf86platformBus.h b/hw/xfree86/common/xf86platformBus.h index 2a90104..317dd24 100644 --- a/hw/xfree86/common/xf86platformBus.h +++ b/hw/xfree86/common/xf86platformBus.h @@ -115,6 +115,9 @@ _xf86_get_platform_device_attrib(struct xf86_platform_device *device, int attrib return xf86_platform_device_odev_attributes(device)->busid; case ODEV_ATTRIB_DRIVER: return xf86_platform_device_odev_attributes(device)->driver; + default: + assert(FALSE); + return NULL; } } @@ -135,6 +138,9 @@ _xf86_get_platform_device_int_attrib(struct xf86_platform_device *device, int at return xf86_platform_device_odev_attributes(device)->major; case ODEV_ATTRIB_MINOR: return xf86_platform_device_odev_attributes(device)->minor; + default: + assert(FALSE); + return 0; } } -- 1.8.3.1
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
