No functional changes, but the documentation explicitly says -1 is returned so let's not return an enum. Returning the enum value is double confusing because it suggests that numeric values may be other members of the enum (hint: they aren't).
Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net> --- libwacom/libwacom.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libwacom/libwacom.c b/libwacom/libwacom.c index 863dc2a..f96bbab 100644 --- a/libwacom/libwacom.c +++ b/libwacom/libwacom.c @@ -1003,7 +1003,7 @@ int libwacom_get_button_led_group (const WacomDevice *device, char button) } } - return WACOM_STATUS_LED_UNAVAILABLE; + return -1; } int libwacom_is_builtin(const WacomDevice *device) -- 2.7.4 ------------------------------------------------------------------------------ What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level. Reveals which users, apps, and protocols are consuming the most bandwidth. Provides multi-vendor support for NetFlow, J-Flow, sFlow and other flows. Make informed decisions using capacity planning reports.http://sdm.link/zohodev2dev _______________________________________________ Linuxwacom-devel mailing list Linuxwacom-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel