typo fix in a comment in a testcase. oh well...
[oweals/busybox.git] / console-tools / setkeycodes.c
index 607b8c78570738553d83849c2801b1b40b44af4b..597272a2feaed1817e942fda727499d48ced32fd 100644 (file)
@@ -9,50 +9,39 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-#include <stdio.h>
-#include <stdlib.h>
-#include <fcntl.h>
-#include <sys/ioctl.h>
-#include "busybox.h"
-
+//#include <sys/ioctl.h>
+#include "libbb.h"
 
 /* From <linux/kd.h> */
 struct kbkeycode {
-       unsigned int scancode, keycode;
+       unsigned scancode, keycode;
 };
 enum {
        KDSETKEYCODE = 0x4B4D  /* write kernel keycode table entry */
 };
 
-extern int
-setkeycodes_main(int argc, char** argv)
+int setkeycodes_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
+int setkeycodes_main(int argc, char **argv)
 {
-       char *ep;
        int fd, sc;
        struct kbkeycode a;
 
-       if (argc % 2 != 1 || argc < 2) {
+       if (!(argc & 1) /* if even */ || argc < 2) {
                bb_show_usage();
        }
 
-       fd = get_console_fd();
+       fd = get_console_fd_or_die();
 
        while (argc > 2) {
-               a.keycode = atoi(argv[2]);
-               a.scancode = sc = strtol(argv[1], &ep, 16);
-               if (*ep) {
-                       bb_error_msg_and_die("error reading SCANCODE: '%s'", argv[1]);
-               }
+               a.keycode = xatou_range(argv[2], 0, 127);
+               a.scancode = sc = xstrtoul_range(argv[1], 16, 0, 255);
                if (a.scancode > 127) {
                        a.scancode -= 0xe000;
                        a.scancode += 128;
                }
-               if (a.scancode > 255 || a.keycode > 127) {
-                       bb_error_msg_and_die("SCANCODE or KEYCODE outside bounds");
-               }
-               if (ioctl(fd,KDSETKEYCODE,&a)) {
-                       bb_perror_msg_and_die("failed to set SCANCODE %x to KEYCODE %d", sc, a.keycode);
-               }
+               ioctl_or_perror_and_die(fd, KDSETKEYCODE, &a,
+                       "can't set SCANCODE %x to KEYCODE %d",
+                       sc, a.keycode);
                argc -= 2;
                argv += 2;
        }