Looks fine.
--
Vadim Zhukov
22 дек. 2014 г. 14:35 пользователь "Alexandr Shadchin" <
alexandr.shadc...@gmail.com> написал:
> Remove function mouse_map. This function is redundant.
>
> OK ?
>
> --
> Alexandr Shadchin
>
> Index: wsmoused.c
> ===================================================================
> RCS file: /cvs/src/usr.sbin/wsmoused/wsmoused.c,v
> retrieving revision 1.33
> diff -u -p -r1.33 wsmoused.c
> --- wsmoused.c 21 Dec 2014 18:23:37 -0000 1.33
> +++ wsmoused.c 21 Dec 2014 23:18:11 -0000
> @@ -218,14 +218,6 @@ mouse_installmap(char *arg)
> return TRUE;
> }
>
> -/* mouse_map : converts physical buttons to logical buttons */
> -static void
> -mouse_map(struct wscons_event *orig, struct wscons_event *mapped)
> -{
> - mapped->type = orig->type;
> - mapped->value = p2l[orig->value];
> -}
> -
> /* terminate signals handler */
> static void
> terminate(int sig)
> @@ -262,7 +254,9 @@ mouse_click(struct wscons_event *event)
> struct timeval max_date;
> struct timeval now;
> struct timeval delay;
> - int i = event->value; /* button number */
> + int i; /* button number */
> +
> + i = event->value = p2l[event->value];
>
> gettimeofday(&now, NULL);
> delay.tv_sec = mouse.clickthreshold / 1000;
> @@ -332,14 +326,11 @@ normalize_event(struct wscons_event *eve
> static void
> treat_event(struct wscons_event *event)
> {
> - struct wscons_event mapped_event;
> -
> if (IS_MOTION_EVENT(event->type)) {
> ioctl(mouse.cfd, WSDISPLAYIO_WSMOUSED, event);
> } else if (IS_BUTTON_EVENT(event->type) &&
> (uint)event->value < MOUSE_MAXBUTTON) {
> - mouse_map(event, &mapped_event);
> - mouse_click(&mapped_event);
> + mouse_click(event);
> }
> }
>
>
>
