Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
---
 libwacom/libwacom-database.c |    9 +++++++--
 libwacom/libwacom.c          |    4 ++--
 libwacom/libwacomint.h       |    1 +
 3 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/libwacom/libwacom-database.c b/libwacom/libwacom-database.c
index fcc9f03..2a0ee19 100644
--- a/libwacom/libwacom-database.c
+++ b/libwacom/libwacom-database.c
@@ -115,6 +115,12 @@ bus_to_str (WacomBusType bus)
        g_assert_not_reached ();
 }
 
+char *
+make_match_string (WacomBusType bus, int vendor_id, int product_id)
+{
+       return g_strdup_printf("%s:%04x:%04x", bus_to_str (bus), vendor_id, 
product_id);
+}
+
 static int
 libwacom_matchstr_to_ints(const char *match, uint32_t *vendor_id, uint32_t 
*product_id, WacomBusType *bus)
 {
@@ -146,7 +152,6 @@ libwacom_parse_stylus_keyfile(WacomDeviceDatabase *db, 
const char *path)
 
        rc = g_key_file_load_from_file(keyfile, path, G_KEY_FILE_NONE, &error);
        g_assert (rc);
-
        groups = g_key_file_get_groups (keyfile, NULL);
        for (i = 0; groups[i]; i++) {
                WacomStylus *stylus;
@@ -302,7 +307,7 @@ libwacom_parse_tablet_keyfile(const char *path)
                        device = NULL;
                        goto out;
                }
-               device->match = g_strdup_printf ("%s:%04x:%04x", bus_to_str 
(device->bus), device->vendor_id, device->product_id);
+               device->match = make_match_string(device->bus, 
device->vendor_id, device->product_id);
                g_free (match);
        }
 
diff --git a/libwacom/libwacom.c b/libwacom/libwacom.c
index 05ac6f1..c16e4b5 100644
--- a/libwacom/libwacom.c
+++ b/libwacom/libwacom.c
@@ -209,7 +209,7 @@ libwacom_new (WacomDeviceDatabase *db, int vendor_id, int 
product_id, WacomBusTy
         return NULL;
     }
 
-    match = g_strdup_printf ("%s:%04x:%04x", bus_to_str (bus), vendor_id, 
product_id);
+    match = make_match_string(bus, vendor_id, product_id);
     device = libwacom_get_device(db, match);
     g_free (match);
 
@@ -343,7 +343,7 @@ libwacom_update_match(WacomDevice *device, WacomBusType 
bus, int vendor_id, int
        device->product_id = product_id;
        device->bus = bus;
        g_free(device->match);
-       device->match = g_strdup_printf("%s:0x%x:0x%x", bus_to_str (bus), 
vendor_id, product_id);
+       device->match = make_match_string(device->bus, device->vendor_id, 
device->product_id);
 }
 
 int libwacom_get_vendor_id(WacomDevice *device)
diff --git a/libwacom/libwacomint.h b/libwacom/libwacomint.h
index 9d5ed7a..960bb3c 100644
--- a/libwacom/libwacomint.h
+++ b/libwacom/libwacomint.h
@@ -124,6 +124,7 @@ void libwacom_update_match(WacomDevice *device, 
WacomBusType bus, int vendor_id,
 
 WacomBusType  bus_from_str (const char *str);
 const char   *bus_to_str   (WacomBusType bus);
+char *make_match_string(WacomBusType bus, int vendor_id, int product_id);
 
 
 #endif /* _LIBWACOMINT_H_ */
-- 
1.7.10


------------------------------------------------------------------------------
Better than sec? Nothing is better than sec when it comes to
monitoring Big Data applications. Try Boundary one-second 
resolution app monitoring today. Free.
http://p.sf.net/sfu/Boundary-dev2dev
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to