Since it could only ever return true, which avoids a totally unnecessary
cleanup path.
Signed-off-by: Daniel Stone <daniel@fooishbar.org>
int i;
CompatInfo info;
GroupCompatInfo *gcm;
- LEDInfo *unbound = NULL, *next;
+ LEDInfo *unbound = NULL;
InitCompatInfo(&info, keymap);
info.dflt.defs.merge = merge;
info.leds = NULL;
}
- if (!BindIndicators(keymap, unbound)) {
- while (unbound) {
- next = (LEDInfo *) unbound->defs.next;
- free(unbound);
- unbound = next;
- }
-
- goto err_info;
- }
+ BindIndicators(keymap, unbound);
ClearCompatInfo(&info, keymap);
return true;
return true;
}
-bool
+void
BindIndicators(struct xkb_keymap *keymap, LEDInfo *unbound)
{
int i;
next = led ? (LEDInfo *) led->defs.next : NULL;
free(led);
}
-
- return true;
}
CopyIndicatorMapDefs(struct xkb_keymap *keymap, LEDInfo *leds,
LEDInfo **unboundRtrn);
-extern bool
+extern void
BindIndicators(struct xkb_keymap *keymap, LEDInfo *unbound);
#endif /* INDICATORS_H */