Revert "Unconstify xkb_rules_names"
authorRan Benita <ran234@gmail.com>
Mon, 7 May 2012 11:54:12 +0000 (14:54 +0300)
committerDaniel Stone <daniel@fooishbar.org>
Tue, 8 May 2012 16:28:49 +0000 (17:28 +0100)
This reverts commit d007cd0a1f3f4b9c927175771ff79aae6fe4ab8b.

This is in fact more restrictive, because it breaks the (common) case
where the strings are const themselved, e.g. "evdev", "us", etc. As is
you must either duplicate the strings or suppress the warnings.

If the user needs to retain the non-const strings, he should instead
just keep them in some other struct and use xkb_rules_names just as
a temporary parameter for xkb_map_new_from_names. Mildly annoying but
acceptable.

include/xkbcommon/xkbcommon.h

index 4eeaaca..4ab9e03 100644 (file)
@@ -111,11 +111,11 @@ typedef uint32_t xkb_led_index_t;
  * should be the primary identifier for a keymap.
  */
 struct xkb_rule_names {
-    char *rules;
-    char *model;
-    char *layout;
-    char *variant;
-    char *options;
+    const char *rules;
+    const char *model;
+    const char *layout;
+    const char *variant;
+    const char *options;
 };
 
 /**